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.

Sign up for GitHub

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

Jump to bottom

fixes bug and updates to new simpy verison #658

Merged

jnsLs merged 2 commits into master from sh/fix_so3

Aug 28, 2024

Collaborator

stefaanhessmann commented Aug 28, 2024

New simpy version caused problems with numpy inputs (ValueError('expecting integer or half-integer, got 0')). Just using the torch tensors instead solves the issue and prevents an unnecessary conversion from torch to numpy
Fixes a typo


          fixes bug and updates to new simpy verison

stefaanhessmann requested a review from jnsLs

August 28, 2024 09:08


          temporary sympy fix

646c2ce

jnsLs approved these changes

View reviewed changes

jnsLs merged commit ea056c6 into master

2 checks passed

jnsLs deleted the sh/fix_so3 branch

August 28, 2024 16:50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet