GEOPY-2884: Change data groups saved for MVI as requested by users

[domfournier]

GEOPY-2884 - Change data groups saved for MVI as requested by users

[Copilot]

Pull request overview

This PR introduces a new GeoH5-saving directive intended to change how inversion model outputs are grouped (for user-requested MVI workflows), and includes a small readability-only formatting change in the vector-model directive.

Changes:

• Reformats the MVI switching conditional in VectorInversion.endIter for readability (no logic change).
• Adds a new SaveModelGroup directive that groups saved FloatData into property groups keyed by component name.
• Exposes SaveModelGroup from simpeg.directives via __init__.py.

Reviewed changes

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

File	Description
simpeg/directives/_vector_models.py	Minor formatting/readability update to the MVI coordinate-switching condition.
simpeg/directives/_save_geoh5.py	Adds SaveModelGroup directive to control property-group naming/grouping.
simpeg/directives/__init__.py	Re-exports SaveModelGroup from the directives package.

Comments suppressed due to low confidence (1)

simpeg/directives/init.py:133

• PR description indicates behavior change for “data groups saved for MVI”, but within this repo the new SaveModelGroup is only defined/exported and not referenced anywhere else. As-is, no grouping behavior will change unless downstream code/workflows switch to this directive. Either update the PR description to reflect that this is a new optional directive, or include the change that wires it into the MVI saving pipeline (if intended).

from ._save_geoh5 import (
    BaseSaveGeoH5,
    SaveDataGeoH5,
    SaveLogFilesGeoH5,
    SaveLPModelGroup,
    SaveModelGroup,
    SaveModelGeoH5,
    SavePGIModel,
    SavePropertyGroup,
    SaveSensitivityGeoH5,
)

---

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