-
Notifications
You must be signed in to change notification settings - Fork 500
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
Support experiment tracking with W&B #213
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.
Thanks for the PR =)
In fact, that was something I wanted to add at some point. Could you also document the use in the readme and update the changelog?
Co-authored-by: Antonin RAFFIN <antonin.raffin@ensta.org>
Also a quick question, I would like to also log raw episodic return instead of the average one |
There are two options:
in both cases, you need a callback and you need to be careful when using multiple envs. |
This is interesting. I guess for Mutil-agent envs, this can be quite tricky. how do you currently log it? |
I tend to use an |
Added the documentation. I tried playing with the callbacks but couldn't quite find a way to pass callback other than overwriting the yaml files... Perhaps the raw stats stuff should be put into a separate PR? |
yes, please |
Co-authored-by: Antonin RAFFIN <antonin.raffin@ensta.org>
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.
minor comments, otherwise LGTM =)
Description
closes #212
Motivation and Context
Types of changes
Checklist:
make format
(required)make check-codestyle
andmake lint
(required)make pytest
andmake type
both pass. (required)Note: we are using a maximum length of 127 characters per line