-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Training playing #4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felimomo sorry i was slow getting to this, edits to the yaml look good! The other changes you mentioned up top:
- added ODE model env caribou_ode.CaribouScipy
- changes to training.py, sb3_train.py
- rebalanced utility fn
- updated yamls
- added simultaneous train script train_algos.sh
- added policy evaluation notebook
I think are already merged? haven't looked over those yet but probably fine, lemme know if there are particular parts of any of that I can look at!
I'll try and be faster at reading these, overall small PRs that change few files are great since easy to review!
Hey @cboettig , I requested the review though we don't have main branch protection - I tried looking into protecting the branch but didn't see the option of require reviews pop up, so we'll have to look at that in more detail. Anyhow, I think it might be good to have a second pair of eyes on the code at this point!
Changes:
caribou_ode.CaribouScipy
training.py
,sb3_train.py
train_algos.sh