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.

Sign up for GitHub

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

Jump to bottom

Add 3D geodesic length measure #10000

Closed
2 tasks done
MV88 opened this issue Feb 28, 2024 · 0 comments · Fixed by #10048, #10104, #10110, #10129 or #10163
Closed
2 tasks done

Add 3D geodesic length measure #10000

MV88 opened this issue Feb 28, 2024 · 0 comments · Fixed by #10048, #10104, #10110, #10129 or #10163
Assignees
@MV88 @ElenaGallo
Labels
3D All issues related to the 3D rendering in CesiumJs Accepted C299- POLITO -2023-MS_DigitalTwin New Feature used for new functionalities
Milestone
2024.01.00

Comments

@MV88
Copy link
Contributor

MV88 commented Feb 28, 2024 •
edited by tdipisa
Loading

Description

Acceptance criteria

  • Add 3D geodesic length measure to the type of 3D measures available
  • lines drawn should be placed on the terrain as a curve, without intersecting it

Other useful information
@MV88 MV88 added New Feature used for new functionalities C299- POLITO -2023-MS_DigitalTwin labels Feb 28, 2024
@MV88 MV88 self-assigned this Feb 28, 2024
@MV88 MV88 added this to the 2024.01.00 milestone Feb 28, 2024
MV88 added a commit to MV88/MapStore2 that referenced this issue Mar 8, 2024
@MV88
Fix geosolutions-it#10000 adding geodesic lines to 3d measures
2fee0d3
@MV88 MV88 mentioned this issue Mar 8, 2024
Merged
12 tasks
@tdipisa tdipisa added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Mar 11, 2024
MV88 added a commit that referenced this issue Mar 20, 2024
@MV88 @allyoucanmap
Fix #10000 adding geodesic lines to 3d measures (#10048)
1fdf6a0 
* Fix #10000 adding geodesic lines to 3d measures

* fix test

* fix test

* fix test

* fixing geodesic label and measured

* updated geodesic length with correct line representation

* also clean up call to computeArea, remvoed uncessary third param

* Update web/client/components/map/cesium/DrawMeasureSupport.jsx

Co-authored-by: stefano bovio <[email protected]>

---------

Co-authored-by: stefano bovio <[email protected]>
@ElenaGallo ElenaGallo self-assigned this Mar 20, 2024
@MV88 MV88 linked a pull request Mar 21, 2024 that will close this issue
Fix #10000 position of total length for geodesic distance #10104
Merged
12 tasks
allyoucanmap pushed a commit that referenced this issue Mar 21, 2024
@MV88
Fix #10000 position of total length for geodesic distance (#10104)
0d941bc
@ElenaGallo ElenaGallo mentioned this issue Mar 22, 2024
Closed
2 tasks
MV88 added a commit to MV88/MapStore2 that referenced this issue Mar 22, 2024
@MV88 @allyoucanmap
Fix geosolutions-it#10000 adding geodesic lines to 3d measures (geoso…
f8f587c 
…lutions-it#10048)

* Fix geosolutions-it#10000 adding geodesic lines to 3d measures

* fix test

* fix test

* fix test

* fixing geodesic label and measured

* updated geodesic length with correct line representation

* also clean up call to computeArea, remvoed uncessary third param

* Update web/client/components/map/cesium/DrawMeasureSupport.jsx

Co-authored-by: stefano bovio <[email protected]>

---------

Co-authored-by: stefano bovio <[email protected]>
MV88 added a commit to MV88/MapStore2 that referenced this issue Mar 22, 2024
@MV88
Fix geosolutions-it#10000 position of total length for geodesic dista…
0d51015 
…nce (geosolutions-it#10104)
@MV88 MV88 mentioned this issue Mar 22, 2024
Merged
@tdipisa tdipisa linked a pull request Mar 22, 2024 that will close this issue
Backport 2024.01.xx - Fix #10000 adding geodesic lines to 3d measures #10110
Merged
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Mar 22, 2024
tdipisa pushed a commit that referenced this issue Mar 22, 2024
@MV88 @allyoucanmap
Backport 2024.01.xx - Fix #10000 adding geodesic lines to 3d measures (…
632688d 
…#10110)

* Fix #10000 adding geodesic lines to 3d measures (#10048)

* Fix #10000 adding geodesic lines to 3d measures

* fix test

* fix test

* fix test

* fixing geodesic label and measured

* updated geodesic length with correct line representation

* also clean up call to computeArea, remvoed uncessary third param

* Update web/client/components/map/cesium/DrawMeasureSupport.jsx

Co-authored-by: stefano bovio <[email protected]>

---------

Co-authored-by: stefano bovio <[email protected]>

* Fix #10000 position of total length for geodesic distance (#10104)

---------

Co-authored-by: stefano bovio <[email protected]>
@tdipisa tdipisa mentioned this issue Mar 25, 2024
Closed
6 tasks
MV88 added a commit to MV88/MapStore2 that referenced this issue Mar 26, 2024
@MV88
Fix geosolutions-it#10000 geodesic lines crash when switching to area…
514b694 
… measurement
@MV88 MV88 mentioned this issue Mar 26, 2024
Merged
12 tasks
allyoucanmap pushed a commit that referenced this issue Apr 4, 2024
@MV88
Fix #10000 geodesic lines crash when switching to area measurement (#…
ad29db2 
…10129)
@tdipisa tdipisa added BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch and removed BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch labels Apr 4, 2024
MV88 added a commit to MV88/MapStore2 that referenced this issue Apr 4, 2024
@MV88
Fix geosolutions-it#10000 geodesic lines crash when switching to area…
d81ba4c 
… measurement (geosolutions-it#10129)
@MV88 MV88 mentioned this issue Apr 4, 2024
Merged
tdipisa pushed a commit that referenced this issue Apr 4, 2024
@MV88
Fix #10000 geodesic lines crash when switching to area measurement (#…
8a865c6 
…10129) (#10163)
@tdipisa tdipisa linked a pull request Apr 4, 2024 that will close this issue
backport 2024.01.xx - Fix #10000 geodesic lines crash when switching to area measurement (#10129) #10163
Merged
offtherailz pushed a commit to offtherailz/MapStore2 that referenced this issue Apr 10, 2024
@MV88 @allyoucanmap @offtherailz
Fix geosolutions-it#10000 adding geodesic lines to 3d measures (geoso…
a809bcc 
…lutions-it#10048)

* Fix geosolutions-it#10000 adding geodesic lines to 3d measures

* fix test

* fix test

* fix test

* fixing geodesic label and measured

* updated geodesic length with correct line representation

* also clean up call to computeArea, remvoed uncessary third param

* Update web/client/components/map/cesium/DrawMeasureSupport.jsx

Co-authored-by: stefano bovio <[email protected]>

---------

Co-authored-by: stefano bovio <[email protected]>
offtherailz pushed a commit to offtherailz/MapStore2 that referenced this issue Apr 10, 2024
@MV88 @offtherailz
Fix geosolutions-it#10000 position of total length for geodesic dista…
071194a 
…nce (geosolutions-it#10104)
offtherailz pushed a commit to offtherailz/MapStore2 that referenced this issue Apr 10, 2024
@MV88 @offtherailz
Fix geosolutions-it#10000 geodesic lines crash when switching to area…
8f8ec7e 
… measurement (geosolutions-it#10129)
@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
Assignees

@MV88 MV88

@ElenaGallo ElenaGallo

Labels
3D All issues related to the 3D rendering in CesiumJs Accepted C299- POLITO -2023-MS_DigitalTwin New Feature used for new functionalities
Projects
None yet
Milestone
2024.01.00
3 participants
@tdipisa @MV88 @ElenaGallo