Skip to content

GEOPY-2015: Add tooltips to missing IP inversion inputs#399

Merged
domfournier merged 3 commits into
release/GA_4.8from
GEOPY-2015
May 8, 2026
Merged

GEOPY-2015: Add tooltips to missing IP inversion inputs#399
domfournier merged 3 commits into
release/GA_4.8from
GEOPY-2015

Conversation

@domfournier
Copy link
Copy Markdown
Collaborator

@domfournier domfournier commented May 6, 2026

GEOPY-2015 - Add tooltips to missing IP inversion inputs

Copilot AI review requested due to automatic review settings May 6, 2026 21:24
@github-actions github-actions Bot changed the title GEOPY-2015 GEOPY-2015: Add tooltips to missing IP inversion inputs May 6, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates multiple .ui.json UI configuration files to improve end-user guidance by adding and refining tooltips for common inversion parameters (e.g., gradient rotation, conjugate-gradient solver settings, and sensitivity-weight thresholding), plus additional context for IP background conductivity inputs.

Changes:

  • Added “Gradient rotation” tooltips describing expected structural orientation inputs.
  • Added CG solver tooltips for max_cg_iterations and tol_cg.
  • Replaced the sens_wts_threshold tooltip text with a more descriptive explanation; added/updated IP background conductivity tooltips.

Reviewed changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
simpeg_drivers-assets/uijson/tipper_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/tdem1d_inversion.ui.json Adds tooltips for CG controls and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/tdem_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/magnetotellurics_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/magnetic_vector_pde_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/magnetic_vector_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/magnetic_scalar_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/joint_surveys_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/joint_petrophysics_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/joint_cross_gradient_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/induced_polarization_3d_inversion.ui.json Adds IP background conductivity tooltip, adds CG/gradient-rotation tooltips, and adds bound tooltips (with wording issues noted).
simpeg_drivers-assets/uijson/induced_polarization_3d_forward.ui.json Adds IP background conductivity tooltip.
simpeg_drivers-assets/uijson/induced_polarization_2d_inversion.ui.json Expands IP background tooltip and adds gradient rotation/CG/sensitivity-threshold tooltips.
simpeg_drivers-assets/uijson/induced_polarization_2d_forward.ui.json Expands IP background tooltip.
simpeg_drivers-assets/uijson/gravity_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/fdem1d_inversion.ui.json Adds tooltips for CG controls and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/fdem_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/direct_current_3d_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/direct_current_2d_inversion.ui.json Adds tooltips for gradient rotation, CG controls, and clarifies sensitivity-weights threshold meaning.
simpeg_drivers-assets/uijson/apparent_conductivity_inversion.ui.json Adds gradient rotation and CG tooltips and clarifies sensitivity-weights threshold meaning.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread simpeg_drivers-assets/uijson/induced_polarization_3d_inversion.ui.json Outdated
Comment thread simpeg_drivers-assets/uijson/induced_polarization_3d_inversion.ui.json Outdated
Comment thread simpeg_drivers-assets/uijson/induced_polarization_3d_inversion.ui.json Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.36%. Comparing base (2ff14af) to head (3d2e19c).

Additional details and impacted files
@@               Coverage Diff               @@
##           release/GA_4.8     #399   +/-   ##
===============================================
  Coverage           90.36%   90.36%           
===============================================
  Files                 110      110           
  Lines                6527     6527           
  Branches              826      826           
===============================================
  Hits                 5898     5898           
  Misses                416      416           
  Partials              213      213           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@domfournier domfournier merged commit b3f9021 into release/GA_4.8 May 8, 2026
16 of 17 checks passed
@domfournier domfournier deleted the GEOPY-2015 branch May 8, 2026 15:48
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