-
Notifications
You must be signed in to change notification settings - Fork 41
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
The merge from Kaolin makes some bugs #10
Comments
same problem raised as you mentioned. Traceback (most recent call last): |
I can share my solution:
check out the git branch to previous commit . |
can you explain me in details. |
i followed the command lines above and it worked . the key point is that the latest version of flexicubes uses kaolin 15 but not adapt their code to kaolin 15's api .so you need to change to previous version( hope this will work for you. |
dear i followed you steps also installed the kaolin==0.13.0 but again same issues i do know what is the problem. |
Dear @SophusDavid |
Apologies for the issues caused by the Kaolin merge. I've added back the previous functions we used to produce the paper's results, which do not depend on Kaolin. The error should be resolved now with the latest commit. |
I download the latest code and run as README.md .
here i think mv,mvp should change to camera (according to the document of Kaolin) But there are some other bugs . As I am not familiar with the entire code , i cant fix them all .So i make this Issue
The text was updated successfully, but these errors were encountered: