Skip to content

Comments

Update wrappers.py#119

Closed
TanmayAmbadkar wants to merge 1 commit intoFarama-Foundation:mainfrom
TanmayAmbadkar:patch-1
Closed

Update wrappers.py#119
TanmayAmbadkar wants to merge 1 commit intoFarama-Foundation:mainfrom
TanmayAmbadkar:patch-1

Conversation

@TanmayAmbadkar
Copy link

Updated vectorized RecordEpisodeStatistics to zero discounted episode returns upon terminations/truncations. Previously, discounted returns would never be zeroed, causing the accumulation of returns without any resetting.

Updated vectorized RecordEpisodeStatistics to zero discounted episode returns upon terminations/truncations. Previously, discounted returns would never be zeroed, causing the accumulation of returns without any resetting.
@ffelten
Copy link
Collaborator

ffelten commented Jul 14, 2025

Hi @TanmayAmbadkar, thanks for the PR! Would you mind having a look at pre-commit?
Also, I think it would be worthy to also update the tests so we know the discounted returns do not get forgotten again.

@TanmayAmbadkar TanmayAmbadkar closed this by deleting the head repository Aug 28, 2025
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.

2 participants