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

Colab results #1470

Closed
Agbeli opened this issue Oct 2, 2023 · 5 comments
Closed

Colab results #1470

Agbeli opened this issue Oct 2, 2023 · 5 comments
Labels

Comments

@Agbeli
Copy link

Agbeli commented Oct 2, 2023

Thank you guys for making this code available to the public. I running demo code and I always get error I would be glad if someone could help out return C.quaternion_to_rotation_matrix(
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
quat_wxyz, order=C.QuaternionCoeffOrder.WXYZ
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
)

@Qu0rise
Copy link

Qu0rise commented Oct 5, 2023

Hello @Agbeli,

Thank you for your engagement with OpenPCDet and for bringing this issue to light.

The error you're encountering is indeed related to a change in the kornia library. Specifically, QuaternionCoeffOrder has been deprecated and removed in versions 0.7.0 and above, as detailed in kornia issue #2599. This alteration is likely the root cause of the error you're experiencing.

🛠 Solution:
You might consider downgrading kornia to a version that still includes QuaternionCoeffOrder. Version 0.6.1 is one such version that you might utilize:

pip install kornia==0.6.1

🔄 Alternative Approach:
Alternatively, if downgrading is not a preferred option, you might consider modifying the code to be compatible with the latest version of kornia. This would involve adjusting the function calls that utilize QuaternionCoeffOrder to align with the new API.

📝 Note:
Ensuring that your environment aligns with the project’s requirements is pivotal to avoid such issues. The requirements.txt or environment.yml file in the project repository might specify the versions of dependencies that the project has been tested with, providing a stable environment for execution.

@yjwong1999
Copy link

Hi @Agbeli

I faced the same problem when running the codes in Colab.
As mentioned by Qu0rise, you might need to consider downgrading kornia
In my case:
pip install kornia==0.5.8

@Agbeli
Copy link
Author

Agbeli commented Oct 10, 2023

Thank you guys

Copy link

This issue is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale label Nov 10, 2023
Copy link

This issue was closed because it has been inactive for 14 days since being marked as stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants