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

Improving geodesic distance for 3D mesurement #10121

Closed
1 of 6 tasks
tdipisa opened this issue Mar 25, 2024 · 1 comment · Fixed by #10130, #10183, #10185 or #10194
Closed
1 of 6 tasks

Improving geodesic distance for 3D mesurement #10121

tdipisa opened this issue Mar 25, 2024 · 1 comment · Fixed by #10130, #10183, #10185 or #10194
Assignees
Labels
3D All issues related to the 3D rendering in CesiumJs Accepted C299- POLITO -2023-MS_DigitalTwin enhancement
Milestone

Comments

@tdipisa
Copy link
Member

tdipisa commented Mar 25, 2024

Description

With #10000 the geodesic distance for 3D measurement has been included. There are a few necessary modifications needed to make that measurement type usable and less ambiguous.

1 - When you draw the first point its position is not fixed in the view and this make the measurement ambiguous: the first point should be anchored at the clicked point

Image

2 - Currently the measurement is drawn for consistency on the ellipsoid at the absolute 0 and this make unclear the measurement position in many cases (specially if you are working at urban level).

measure_2.mp4

The position of the measurement should be preserved as drawn by the user but it should be clearly indicated somehow that it is referencing a distance calculated at the absolute 0 (maybe we can do this info in tooltip and use guide).

Image

What kind of improvement you want to add? (check one with "x", remove the others)

  • Minor changes to existing features
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Other useful information

@tdipisa tdipisa added this to the 2024.01.00 milestone Mar 25, 2024
@tdipisa tdipisa added C299- POLITO -2023-MS_DigitalTwin BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch labels Mar 25, 2024
MV88 added a commit to MV88/MapStore2 that referenced this issue Mar 26, 2024
MV88 added a commit that referenced this issue Apr 4, 2024
* Fix #10121 moving the rendered measure as the normal one

* review line position

---------

Co-authored-by: allyoucanmap <[email protected]>
@ElenaGallo ElenaGallo self-assigned this Apr 4, 2024
MV88 added a commit to MV88/MapStore2 that referenced this issue Apr 9, 2024
…ne (geosolutions-it#10130)

* Fix geosolutions-it#10121 moving the rendered measure as the normal one

* review line position

---------

Co-authored-by: allyoucanmap <[email protected]>
allyoucanmap added a commit that referenced this issue Apr 10, 2024
…0183)

* Fix #10121 moving the rendered measure as the normal one

* review line position

---------

Co-authored-by: allyoucanmap <[email protected]>
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Apr 10, 2024
offtherailz pushed a commit to offtherailz/MapStore2 that referenced this issue Apr 10, 2024
…ne (geosolutions-it#10130)

* Fix geosolutions-it#10121 moving the rendered measure as the normal one

* review line position

---------

Co-authored-by: allyoucanmap <[email protected]>
@ElenaGallo
Copy link
Contributor

@MV88 the height of the geodetic measurement changes when you switch to area and angle measurement. (on DEV and QA)

geodesic.mp4

MV88 added a commit to MV88/MapStore2 that referenced this issue Apr 10, 2024
@ElenaGallo ElenaGallo added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Apr 11, 2024
MV88 added a commit to MV88/MapStore2 that referenced this issue Apr 12, 2024
@ElenaGallo ElenaGallo removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Apr 12, 2024
@tdipisa tdipisa added the 3D All issues related to the 3D rendering in CesiumJs label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment