-
Notifications
You must be signed in to change notification settings - Fork 64
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
Batch #225
Batch #225
Conversation
DerThorsten
commented
Nov 16, 2021
- added batch API for pybox2d
- this is not fitting very well to the existing batch api since we also pass an array of colors
- might need refactoring
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super nice!! Thanks :)
I haven't tested yet, but I have some comments:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Looking pretty good :)
We should document those new methods in a follow-up PR |
Co-authored-by: martinRenou <martin.renou@gmail.com>
Co-authored-by: martinRenou <martin.renou@gmail.com>
Co-authored-by: martinRenou <martin.renou@gmail.com>
@martinRenou the "Test / build" part of the CI fails with some python 3.10 related error in |
I can also update the docs in this if you prefer that =) |
It's fine in another PR :) |
We'll need to update jupyter-packaging I guess... But let's do that in a separate PR too. Thanks! |