Reset Camera Closest is broken

Zoom close to data is way broken. Here is how to replicate it:

• Master, Linux, builtin server.
• Make ParaView's renderview window about twice as wide as it is tall. In File/ Save Screenshot, I have 1842X897.
• Sources/ Cylinder. Apply.
• Reset. +Y.
• Adjust Camera. Elevation = 35. Azimuth = 35.
• Reset Camera Closest.

added priority:required label

@paul.lafoix

changed title from zoom close to data is broken to Reset Camera Closest is broken

Another example of this being broken is replicated as follows:

• Master, Linux, builtin server.
• Sources/ Sphere. Theta Resolution - 64, Phi Resolution - 64. Apply.
• Reset Camera Closest.
• Rotate the Sphere.
• Reset Camera Closest.

The sphere will zoom in or out. What I would expect is that the sphere will be the same size no matter how it is oriented, which is true with "Reset".

Hi, I cannot reproduce the first issue :

• master builtin, size 1555*607
• Cylinder, reset +Y, Elevation 35, azimuth 35
• Reset Camera Closest :

For the second one, I investigated. The bounds passed to the ResetCameraScreenSpace function are the same, but as the ViewTransform of the camera is modified after rotating the camera, the method GetCompositeProjectionTransformMatrix used in the method vtkRenderer::WorldToView() returns a different matrix.

That's why the screen space bounding box is different but should be the same (in the case of a perfectly round sphere...). I would need to investigate more

any ideas @cory.quammen ?

changed milestone to %5.11 (Spring 2022)

same here, unable to reproduce the issue. The issue in your comment is indeed present but this is exactly the situation @paul.lafoix mentionned, does not seems a big issue though.

The initial, reported bug is fixed. I also cannot replicate.

• MacOS, 5.10.0-RC2, builtin server.

Hey @paul.lafoix @mwestphal This one appears to be trivial enough that we can just close it until we have a customer complain. Shall we do that?

I agree.

closed