Replies: 1 comment
-
|
Ah nice catch, no this is not explicitly intended but i am not sure if it causes any difference in intended logic. The main reason i use the jumanji one is to ensure a consistent API especially considering the next_obs info relevant for truncated environment bootstrapping. It would be worth checking if turning off the jumanji API has any impact on the performance and logic. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Afaict, navix autoresets during its internal step-function (https://github.com/epignatelli/navix/blob/5fc4975a9671e4ebbc101a1eaabc72364c0e33d1/navix/environments/environment.py#L162-L170), but in Stoix', navix environments are also wrapped with Jumanji's AutoResetWrapper. Is this intended behaviour?
Beta Was this translation helpful? Give feedback.
All reactions