GEOPY-2630: Tile estimation creates a new group, but should use the one you give it.

[domfournier]

GEOPY-2630 - Tile estimation creates a new group, but should use the one you give it.

[Copilot]

Copilot review overview

🔵 Needs a closer look

Restore coverage for default output behavior and update the return annotation for UIJsonGroup.

Review effort: Lite
Findings: None

What changed in this PR

Adds optional output-group support to tile estimation while simplifying initialization and updating integration coverage.

Changes:

• Exposes an optional output-group selector.
• Makes tile parameters mutable and avoids eager inversion setup.
• Updates grouped-output tests.

File	Summary
tests/​run_tests/​driver_tile_estimator_test.py	Tests explicit output-group handling.
simpeg_drivers/​utils/​tile_estimate.py	Supports mutable parameters and grouped output.
simpeg_drivers-assets/​uijson/​tile_estimator.ui.json	Adds the optional output-group field.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.48%. Comparing base (24b4b9e) to head (25faafb).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop     #481      +/-   ##
===========================================
+ Coverage    90.45%   90.48%   +0.02%     
===========================================
  Files          111      111              
  Lines         7105     7105              
  Branches       880      880              
===========================================
+ Hits          6427     6429       +2     
+ Misses         458      456       -2     
  Partials       220      220              

Files with missing lines	Coverage Δ
simpeg_drivers/utils/tile_estimate.py	88.97% <100.00%> (+1.47%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.