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

(Outputted results will be changed) output whole results #354

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

YuriCat
Copy link
Contributor

@YuriCat YuriCat commented Apr 19, 2024

If results for evaluation episodes that were not completed by the end of the epoch are truncated, not only will the number of results displayed be reduced, but there will be some bias on the results by the fact that longer episodes are more likely to be ignored.

While there is a downside to the results displayed not matching the epoch, the concern about results being discarded is more significant in practical development.

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