Skip to content

Resolve "Add 3D measure tool to Cesium"

Corentin Junod requested to merge 208-add-3d-measure-tool-to-cesium into main

Description

Add 3D measure tool to Cesium

Job done

  • Add length / area information on shapes in 2D view

  • Handle drawing in 3D view

    • Points
    • Polylines
    • Polygons
    • Circles
    • Squares
    • Rectangles
    • Freehand lines
    • Freehand polygones
  • Add 3D view shapes in the drawing tool list

  • Add length / area information on shapes in 3D view

  • Add full support for (de)serialization

  • Proper handling of components (handle the case if the 2Dmap / 3Dmap is not there + avoid relying on the instancing order)

  • Ensure no TODO are left in the code

Definition of Done

For the Developer:

  • Code compiles and conforms to defined coding standards (eslint, tsc, prettier).
  • If necessary (at the developer's discretion), unit tests have been added for critical parts.
  • All unittests run without error.
  • Sonar-Scan returns no new issues or security hotspots.
  • Documentation has been updated where necessary (see documentation repository).
  • Merge-Request contains a few explanations of what was done.
  • A Reviewer has been assigned to the Merge-Request.

For the Reviewer:

  • New code was reviewed.
  • If comments could be added on unclear code, the developer has been informed.
  • If unitests could be added in certain places, the developer has been informed.
  • Merge-Request was merged on main branch.
  • All pipelines work.
  • The bugfix was tested on demo environment.
  • Application performance has not been degraded by the modifications.

Authors

Co-authored-by: Dolpic dolpic@outlook.com

Closes #208 (closed) and #214 (closed)

Edited by Stéphane Malta e Sousa

Merge request reports