GEOPY-2739: Accept BaseUIJson in the start of driver

[domfournier]

GEOPY-2739 - Accept BaseUIJson in the start of driver

[Copilot]

Pull request overview

This PR updates geoapps-utils to work with geoh5py.ui_json.UIJson as the primary ui.json representation (instead of InputFile/BaseUIJson), including allowing drivers to start from a UIJson instance and updating related templates and tests.

Changes:

• Switch driver startup and UIJsonGroup execution flow to use UIJson objects directly.
• Refactor Options to derive/serialize UIJson via a default template (base.ui.json) and adjust tests accordingly.
• Update the base ui.json asset schema for out_group and pin geoh5py to a GEOPY-2739 revision.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
geoapps_utils/base.py	Refactors Driver.start and Options to operate around UIJson, adds deprecation path for InputFile, and updates serialization/out_group saving.
geoapps_utils/run.py	Updates run_uijson_group and workspace/ui.json copy helpers to use UIJson.read/from_dict/write.
geoapps_utils-assets/uijson/base.ui.json	Updates the base template to represent out_group as a structured group parameter object.
tests/uijson_run_test.py	Updates test setup to build and write a UIJson file directly and adjusts expected filenames.
tests/driver_test.py	Aligns tests with new UIJson-based serialization behavior and updated exception types.
tests/dataclass_test.py	Loosens flatten equality assertions to avoid direct geoh5 comparisons and updates UIJson writing callsites.
pyproject.toml	Pins geoh5py dependency to rev = "GEOPY-2739".

---

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

[codecov]

Codecov Report

❌ Patch coverage is 87.05882% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.68%. Comparing base (f0289a8) to head (51c1155).
⚠️ Report is 29 commits behind head on feature/uijson.

Files with missing lines	Patch %	Lines
geoapps_utils/base.py	86.27%	3 Missing and 4 partials ⚠️
geoapps_utils/run.py	81.25%	1 Missing and 2 partials ⚠️
geoapps_utils/param_sweeps/generate.py	91.66%	1 Missing ⚠️

Additional details and impacted files

@@                Coverage Diff                 @@
##           feature/uijson     #190      +/-   ##
==================================================
+ Coverage           85.12%   85.68%   +0.55%     
==================================================
  Files                  19       18       -1     
  Lines                1163      908     -255     
  Branches              151      114      -37     
==================================================
- Hits                  990      778     -212     
+ Misses                134       95      -39     
+ Partials               39       35       -4     

Files with missing lines	Coverage Δ
geoapps_utils/param_sweeps/driver.py	81.60% <100.00%> (-0.21%)	⬇️
geoapps_utils/param_sweeps/generate.py	78.94% <91.66%> (+4.58%)	⬆️
geoapps_utils/run.py	90.99% <81.25%> (-1.61%)	⬇️
geoapps_utils/base.py	83.53% <86.27%> (-8.18%)	⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:

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

[MatthieuCMira]

I have the feeling the tests were supress without being replaced.

As this is at the root of all our apps, it shoulkd be heavily unitested right?

!!

[MatthieuCMira]

failueres with scigeoh5.
The solution is to open everything in the same workspace.

[MatthieuCMira]

Still, when the test are passing, a very low coverage:

...

As I said before, we want to test our base package a lot!

[MatthieuCMira]

It's getting good.
Few comments, mostly questions

[MatthieuCMira]

good for me

[MatthieuCMira]

good to go