-
Notifications
You must be signed in to change notification settings - Fork 516
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
Add a callback to log raw stats #216
Conversation
@araffin also suggested I do the same for |
ah no, there was some misunderstanding on that. |
@vwxyzjn I had to do a PR to your fork as I couldn't push to it :/ |
Fixes and use tensorboard output directly
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.
Could you check what I did?
if you add test and update changelog, we are good to go ;)
Added a test case and updated the changelog :) |
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.
LGTM =)
Awesome. Fell free to merg |
Awesome. Feel free to merge |
This PR adds a callback to log raw (unaveraged) stats such as
episodic_length
,episodic_returns
. The testing command ispython train.py --algo ppo --env CartPole-v1 -params callback:'"utils.callbacks.RawStatisticsCallback"' --track
and it produces this runMotivation 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