Skip to content

Conversation

@domfournier
Copy link
Contributor

@domfournier domfournier commented Jan 19, 2026

GEOPY-2449 - Create UI and Driver for the classification of EM anomalies

@github-actions github-actions bot changed the title Geopy 2449 GEOPY-2449: Create UI and Driver for the classification of EM anomalies Jan 19, 2026
@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

❌ Patch coverage is 95.60440% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.33%. Comparing base (3a0ee39) to head (1e7f0aa).

Files with missing lines Patch % Lines
geoapps_utils/utils/transformations.py 95.18% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #178      +/-   ##
===========================================
+ Coverage    86.09%   86.33%   +0.24%     
===========================================
  Files           19       19              
  Lines         1100     1120      +20     
  Branches       148      150       +2     
===========================================
+ Hits           947      967      +20     
  Misses         114      114              
  Partials        39       39              
Files with missing lines Coverage Δ
geoapps_utils/base.py 91.42% <ø> (ø)
geoapps_utils/driver/data.py 100.00% <ø> (ø)
geoapps_utils/driver/driver.py 100.00% <ø> (ø)
geoapps_utils/driver/params.py 75.73% <ø> (ø)
geoapps_utils/modelling/plates.py 100.00% <ø> (ø)
geoapps_utils/param_sweeps/driver.py 81.81% <ø> (ø)
geoapps_utils/param_sweeps/generate.py 74.35% <ø> (ø)
geoapps_utils/run.py 92.59% <ø> (ø)
geoapps_utils/utils/conversions.py 100.00% <ø> (ø)
geoapps_utils/utils/formatters.py 100.00% <ø> (ø)
... and 9 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

MatthieuCMira
MatthieuCMira previously approved these changes Jan 20, 2026
Copy link
Contributor

@MatthieuCMira MatthieuCMira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems good for me

Convert Cartesian coordinates (x, y, z) to polar coordinates defined as
(distance, azimuth, height).
Distances are relative to the origin, azimuth angles are
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The origin is always 0,0,0.
Don't we want sometimes to change that?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good point. Let's do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants