-
Notifications
You must be signed in to change notification settings - Fork 27
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
A Pytorch extension for your occupancy grid and volume rendering modules #6
Comments
Hi there! Thanks again for taking the time and for the feedback! Currently I am also unhappy with how entangled the OccupancyGrid and VolumetricRendering is with the rest of the dependencies. Also if you need any help with the refactoring or with the code migration let me know. |
Hi, thanks for you feedback. I've release my version of refactored code on github HERE. |
Hi @jingsenzhu, thanks for your work! Does your refactored code still need PyTorch to be installed with CXX_ABI=1? |
Hi @s-esposito , can you describe your problem in detail? My refactored code should be able to be installed with the default Pytorch from PIP or Conda. |
Hi @jingsenzhu, thanks for the answer. |
I don't think my code has this requirement, for I've successfully run my code under a Pytorch directly installed from PIP without any other constraints. Tell me if you meet other problems when you install my code. |
Thank you again for your great work!
In your implementation of Permutohedral Encoding, you implement it as a Pytorch CUDA module so it's scalable and easy to migrate to other NeRF and SDF projects. However, I found that you implemented the Permuto SDF modules as pure CUDA classes requiring a lots of dependency libraries ,which I struggled in installing on my system without using a docker :( . Therefore, for code migration needs, I slightly modified your codes which removed most of the dependencies (of course, Pytorch and CUDA are still needed :) , making it into a installable Pytorch module.
If you agree, I'll release my modified codes on github. Thank you very much :D
The text was updated successfully, but these errors were encountered: