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

Fix docstrings in StateWrapper, CarWrapper, and PhysicsWrapper #35

Open
mrkvicka22 opened this issue Jan 4, 2022 · 0 comments
Open

Comments

@mrkvicka22
Copy link
Contributor

mrkvicka22 commented Jan 4, 2022

More specifically, the methods StateWrapper.format_state(), PhysicsWrapper._encode() and PhysicsWrapper._encode() state that they return strings when in fact they return python lists. Note: Other methods that deal with decoding/encoding most likely also have faulty docstrings.

Maybe they should be left as numpy arrays for performance reasons unless it makes sense to keep them as lists.

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

No branches or pull requests

1 participant