-
Notifications
You must be signed in to change notification settings - Fork 16
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 camera intrinsic [cx] and [cy] is useless in this demo. #4
Comments
It is used in the intrinsic, see the K matrix in this function, for instance: render_depthmap_example/visualization.py Line 133 in dbd53b3
Or what do you mean exactly? |
I mean , the (.capture_depth) or (capture_screen) is still using the intrinsic [ width, height, fx, fy , width/2 , height/2], not the [ width, height, fx, fy , cx , cy]. Here is the same issue : [https://github.com/isl-org/Open3D/issues/3079] |
Do you mean this:
|
I mean the following code block:
the flag "allow_arbitrary" doesn't change the camera intrinsic value about "cx" and "cy". As a result, the function (.capture_depth) 或 (capture_screen) is still using the intrinsic [ width, height, fx, fy , width/2 , height/2]. |
Hi @pablospe ! I met with the same problem. If I change the It means that the visualizer of open3D still uses the intrinsic [ width, height, fx, fy , width/2 , height/2] instead of [ width, height, fx, fy , cx , cy] that I specify. |
One would need to debug the Open3D viewer to find the problem and see where is set to I had some problems with Open3D viewer so I tried to replicate this code using pyvista (VTK backend), in this branch: |
Another alternative: |
Thank you for your prompt response. I will give it a try ! 😃 |
The camera intrinsic [cx] and [cy] is useless in this demo.
The text was updated successfully, but these errors were encountered: