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

Rotating camera with point cloud data is very confusing #40530

Closed
saberraz opened this issue Dec 9, 2020 · 8 comments · Fixed by #45979
Closed

Rotating camera with point cloud data is very confusing #40530

saberraz opened this issue Dec 9, 2020 · 8 comments · Fixed by #45979
Labels
3D Relates to QGIS' 3D engine or rendering Bug Either a bug report, or a bug fix. Let's hope for the latter! Feedback Waiting on the submitter for answers Point Clouds stale Uh oh! Seems this work is abandoned, and the PR is about to close.

Comments

@saberraz
Copy link
Contributor

saberraz commented Dec 9, 2020

...as it uses the terrain for the point of reference.

@saberraz saberraz added Bug Either a bug report, or a bug fix. Let's hope for the latter! Point Clouds labels Dec 9, 2020
@roya0045
Copy link
Contributor

roya0045 commented Dec 9, 2020

@saberraz wasn't that always the case with 3D? and is this even a bug or just a UI/UX issue?

@saberraz
Copy link
Contributor Author

saberraz commented Dec 9, 2020

It is particularly an issue with point cloud, as users usually will end up with a flat terrain.

@uclaros
Copy link
Contributor

uclaros commented Dec 9, 2020

This is unrelated to point cloud data.
I also find camera movement in 3D both bad UX and buggy.

  • Normal pan is good and intuitive.
  • Middle mouse drag to rotate has double-sensitivity on the Vertical axis than on the Horizontal making it very unstable to use, almost unpredictable. This is probably a bug, I can't imagine why it could be that way on purpose. edit: Fix 3d camera yaw and pitch sensitivity #40680 Fixes this.
  • Middle mouse drag to rotate rotates the view around some indistinguishable point (the middle of the scene?) while intuition says it should rotate around the terrain point where the cursor was when clicked (just like google earth does). Showing a marker on the center of rotation would be a huge UX improvement too!
  • Ctrl + Click drag to rotate should keep the camera position and rotate around it ... well, tbh, I don't even know what exactly it does right now!! update: upgrading from qt 5.14.2 to 5.15.2 resolved this!

@uclaros
Copy link
Contributor

uclaros commented Dec 19, 2020

* Ctrl + Click drag to rotate should keep the camera position and rotate around it ... well, tbh, I don't even know what exactly it does right now!!

Apparently, upgrading from qt 5.14.2 to qt 5.15.2 and rebuilding resolved the drunken camera movement I was experiencing!!
Now the only issue with ctrl+click drag to rotate is that the camera does not stay 100% put while rotating, but it slightly moves too, because instead of directly setting the camera's pitch and yaw, its location is recalculated from the center of the scene.

@jedfrechette
Copy link

  • Middle mouse drag to rotate rotates the view around some indistinguishable point (the middle of the scene?) while intuition says it should rotate around the terrain point where the cursor was when clicked (just like google earth does). Showing a marker on the center of rotation would be a huge UX improvement too!

Plus one for rotating around the position of the cursor when the tumble operation was initiated. Currently, rotation seems to occur around the center of the viewport. That means that if you want to rotate around a specific point you first need to move that point to the center of the viewport, which is inevitably imprecise, then initiate the tumble. Rotating around the cursor position would decrease that operation to a single step and make it more precise.

@jonnyforestGIS
Copy link
Contributor

Hi @saberraz, I don't know if this is related but, when the point cloud is measured in absolute values, you need to do the offset to the minimum value of Z to "normalize". This improves a lot of navigation and search.

If a point cloud is georeferenced, you are dealing with the sea level distance so this will create a distance between ground and point of view of the point cloud, So you need to add an offset with a negative value using the minimum Z. The plus of this is that you can see the scale with real Z values of the point cloud objects.

In point cloud that is already normalized in relative values, you don't need to do the offset.

@gioman gioman added the 3D Relates to QGIS' 3D engine or rendering label Feb 18, 2021
@wonder-sk
Copy link
Member

This should get fixed with #45979

@gioman gioman added the Feedback Waiting on the submitter for answers label Dec 15, 2021
@github-actions
Copy link

The QGIS project highly values your report and would love to see it addressed. However, this issue has been left in feedback mode for the last 14 days and is being automatically marked as "stale".
If you would like to continue with this issue, please provide any missing information or answer any open questions. If you could resolve the issue yourself meanwhile, please leave a note for future readers with the same problem and close the issue.
In case you should have any uncertainty, please leave a comment and we will be happy to help you proceed with this issue.
If there is no further activity on this issue, it will be closed in a week.

@github-actions github-actions bot added the stale Uh oh! Seems this work is abandoned, and the PR is about to close. label Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3D Relates to QGIS' 3D engine or rendering Bug Either a bug report, or a bug fix. Let's hope for the latter! Feedback Waiting on the submitter for answers Point Clouds stale Uh oh! Seems this work is abandoned, and the PR is about to close.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants