Skip to content

Comments

Initialize speed and position with float constants#658

Merged
eleurent merged 1 commit intoFarama-Foundation:masterfrom
risteon:master
Aug 29, 2025
Merged

Initialize speed and position with float constants#658
eleurent merged 1 commit intoFarama-Foundation:masterfrom
risteon:master

Conversation

@risteon
Copy link
Contributor

@risteon risteon commented Aug 27, 2025

Ensure that speed and position are initialized using floating-point values. This prevents vectorized environment wrappers from inferring an integer dtype when a single environment resets, which could otherwise cause loss of precision.

Ensure that speed and position are initialized using floating-point values.
This prevents vectorized environment wrappers from inferring an integer dtype
when a single environment resets, which could otherwise cause loss of precision.
@risteon
Copy link
Contributor Author

risteon commented Aug 28, 2025

Amended vec env test so that envs without a duration config will also terminate

@eleurent eleurent merged commit 75342a1 into Farama-Foundation:master Aug 29, 2025
5 checks passed
@eleurent
Copy link
Collaborator

Thanks a lot for the change and for fixing the test!

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