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

Successful neuralsim and nr3d_lib version for street-view surface reconstruction #17

Open
haoyuhsu opened this issue Sep 27, 2024 · 7 comments

Comments

@haoyuhsu
Copy link

Hi everyone,

I have tested multiple versions of neuralsim and nr3d_lib on street-view surface reconstruction and encountered some issues due to version mismatches between the two packages. These issues include: (1) an "unrecognized key" error when loading the preprocessed dataset, and (2) excessively long compilation times for nr3d_lib.

After several attempts, I found that the combination of commit d0bee89 in neuralsim and commit b3d9627 in nr3d_lib worked for me.

I hope this helps anyone who might be stuck running the repo!

@lhp121
Copy link

lhp121 commented Dec 1, 2024

@haoyuhsu Hello, may I ask which version of neuralsim you are using? Is it neuralsim-dev, neuralsim-0.5.1, or neuralsim-pre-releases? Thank you.

@haoyuhsu
Copy link
Author

haoyuhsu commented Dec 2, 2024

@lhp121 I am not sure what the neutralism versions mean. I simply checkout to the commit point b3d9627 in nr3d_lib.

@lhp121
Copy link

lhp121 commented Dec 2, 2024

@haoyuhsu Okay, thank you very much! I have now installed the third-party libraries and can start training. I found the version of the neuralsim code you mentioned. However, I'm currently facing this issue that I can't resolve, and I wonder if you've encountered it before? AssertionError: If capturable=True, params and state_steps must be CUDA tensors. (Also, I'd like to ask if you are in this research direction, and can we exchange contact information to have more discussions?)

@haoyuhsu
Copy link
Author

haoyuhsu commented Dec 5, 2024

@lhp121 Do you remember to also checkout to the commit point d0bee89 in neuralsim? Unfortunately, I am not an expert in this research direction; I just use this repo as a surface reconstruction toolkit in my project.

@lhp121
Copy link

lhp121 commented Dec 5, 2024

@haoyuhsu Yes, I have switched to the commit of neuralsim that you successfully ran. The previous error "If capturable=True, params and state_steps must be CUDA tensors." The author said it was due to the PyTorch version, and I resolved this issue by switching PyTorch from 1.13.0 to 2.0.0. However, when training with the data provided by the author, a new error occurred at 3%: y = ((y - prev) / (cdf_y[frame_ind, h] - prev) + h) / res_y Runtime Error: CUDA error: device-side assert triggered. I am currently debugging and making modifications, and I greatly appreciate your help and response!

@haoyuhsu
Copy link
Author

haoyuhsu commented Dec 5, 2024

@lhp121 For my project, I need to re-run the data preprocessing. I suspect there may be some discrepancies between the processed data provided by the author and the output generated directly by the code.

@lhp121
Copy link

lhp121 commented Dec 5, 2024

@haoyuhsu Okay, thank you very much! Your guess makes a lot of sense because the data I used for training was directly downloaded from the authors without any preprocessing of my own. I feel that there are no issues with the logic of the code, and I am quite concerned about making changes to the code that might affect subsequent training. I will try preprocessing the data myself!

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

No branches or pull requests

2 participants