Viewer and camera height and width issue

[arunmallya]

Describe the bug
[ViewControl] ConvertFromPinholeCameraParameters() failed because window height and width do not match.
Setting the view to a specified camera fails even when the window height and width of the viewer and the camera match.

To Reproduce
https://gist.github.com/arunmallya/41374d7eed6a52923b6ca2eb864b420a
Data file: https://uofi.box.com/s/spqlqx6c6tggpedv6ehs6muz0deyogy0

Environment (please complete the following information):

• OS: Ubuntu 16.04
• Python version: 3.5.2
• Open3D version: 0.7.0.0
• Is this remote workstation?: no
• How did you install Open3D?: pip

[syncle]

Could you check these lines? You may need to replace

K = [[focal * width, 0, width / 2],
     [0, focal * width, height / 2],
     [0, 0, 1]]

as

K = [[focal * width, 0, width / 2 - 0.5],
     [0, focal * width, height / 2 - 0.5],
     [0, 0, 1]]

[germanros1987]

@errissa we will take care of this after the 0.11 release...

[eriksandstroem]

@germanros1987 So there is still no solution to this problem? Adding -0.5 as suggested above did not work for me. I am trying to run the example script customized_visualization.py

Edit: After lots of gruesome pain I was able to get it to work. The key is to look at the code in ViewControl.cpp located at Open3D/cpp/visualization/visualizer/ViewControl.cpp.

All conditions need to be fulfilled in order to make it work.

[Zumbalamambo]

@germanros1987 so curious... Stuck with this issue for a week

[gonultasbu]

@tfy14esa thank you for your effort!

Adding -0.5 as suggested above did not solve the problem for me either. Turns out the window size also has to match with the intrinsic parameters and resolution. As follows:

viewer.create_window(width=width, height=height)

[hanjiangrui]

@tfy14esa thank you for your effort!

Adding -0.5 as suggested above did not solve the problem for me either. Turns out the window size also has to match with the intrinsic parameters and resolution. As follows:

viewer.create_window(width=width, height=height)
it's right

[YuDongSir]

I got the final solution:
just search
"
ctr.convert_from_pinhole_camera_parameters(
glb.trajectory.parameters[glb.index])
"
and add an argument
"
ctr.convert_from_pinhole_camera_parameters(
glb.trajectory.parameters[glb.index], True)
"
it will allow an arbitrary windows size, which can be seen easily in the source code.

[theNded]

Should have been supported in #2564.

[RuipingL]

in current version, you should set the parameter allow_arbitrary=True, i.e.,
view_control.convert_from_pinhole_camera_parameters(cam_params, allow_arbitrary=True)

[Paullfx]

in current version, you should set the parameter allow_arbitrary=True, i.e., view_control.convert_from_pinhole_camera_parameters(cam_params, allow_arbitrary=True)

This works! Saves my day :)

[zhaoruifeng1211]

in current version, you should set the parameter allow_arbitrary=True, i.e., view_control.convert_from_pinhole_camera_parameters(cam_params, allow_arbitrary=True)

Thank you, this is the best solution!