-
Notifications
You must be signed in to change notification settings - Fork 99
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
Run demo.py, while only loops twice and ends #49
Comments
The most likely reason is that the input data was not found. Can you double check that the video_stream or image_stream is actually loading the images? |
Hello @lahavlipson I'm not sure if changing the long long type is causing it to not work properly, or if DPVO is not compatible with Windows. Can you identify the cause of the problem or test compatibility with Windows? Thank you very much! |
I wasn't familiar with this issue, thanks for letting me know; I will investigate. |
Hello, @lahavlipson |
I did go further and with me it was exiting on this line (I'm also on windows and had to compile hanging the source from long to int64_t (i think it was this): Line 188 in 5833835
it stops running wihtout any error. |
@carlosedubarreto Yes, you are correct. It actually exited at this line: Line 60 in 5833835
motion_probe()->reproject()->pops.transform() |
Even more specific, while debugging, the error happens at exactly this point
If i watch this part, the process crashes also while debugging giving no error. I'm starting to think it might be something related with pytorch. I plan to test on other pytorch version. Since I was using to process wham, I'm with pytorch 1.11 cuda 11.3 |
@carlosedubarreto I captured the entire process of error occurrence using looking from bottom to top:
|
great finding @psiydown . so probably the guy that is causing problems is the compilation of lietorch... |
@carlosedubarreto |
I will investigate this issue. But if the issue is lietorch specifically, then in the meantime you can replace the lietorch.SE3 type with the SE3 type from pypose. The data types are virtually identical, but the latter is written entirely in python. You should only need to replace SE3.exp -> pp.Exp and .inv() -> .Inv() |
@lahavlipson , I made a lot of changes to use the pypose library, but it didnt work too. thanks for the help |
@lahavlipson @carlosedubarreto Regarding this issue, lietorch cannot be used on Windows 10.He suggest "managed to fix this on Windows by making all the CUDA kernels in |
Another great finding @psiydown . |
@psiydown I was able to make it work. I changes the files from long to int64_t and also removed the "const" from the lietorch_gpu.cu Here are the files I changed to be able to compile and not crash. |
@carlosedubarreto Next, I plan to implement real-time camera WHAM inference. If you are interested, if you encounter any problems during the implementation process, we can communicate via email and exchange opinions. My email: [email protected] |
The result was that the file was not saved and there were no error messages. What is the reason?
Tracked to, second loop, \dpvo\dpvo.py:
if self.n>0 and not self.is_ Initialized:
Print out: n=1, is_ Initialized=False, motion_probe()=NULL, The return has ended.@lahavlipson
The text was updated successfully, but these errors were encountered: