Skip to content
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

Modify evaluation score processing #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gorisanson
Copy link
Contributor

@gorisanson gorisanson commented May 15, 2019

So that it does not record NaN. Now gif is always generated after an evaluation.

For the case you do not want this pull request, I divided this pr from the reward clipping one #3.

I think the gif file size could be a problem. When I tested, if the agent got stuck in a loop, the size of a gif file created is about 180 MB big. In worst case, the disk usage for gif files would be about 150 * 180 MB = 27 GB. And I think maybe in average case it is about 16GB (On my training, 78 of 150 was the unfinished first evaluation game case, and rest 72 was the finished case whose gif files have disk usage about 1.7GB. So, 78 * 180 MB + 1.7 GB = 14GB + 1.7 GB = about 16 GB).

It would be not a problem when a machine has a sufficient disk space. Considering tensorflow save model files have disk usage about 12GB, over 28GB disk space is required to run the train code.

So that it does not record NaN.
Now gif is always generated after an evaluation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant