Skip to content

Commit 949e767

Browse files
authored
Merge pull request #326 from MiraGeoscience/GEOPY-2624
GEOPY-2624: check for MVI Simpeg groups and raise error
2 parents 86d4970 + 19e953d commit 949e767

1 file changed

Lines changed: 14 additions & 2 deletions

File tree

simpeg_drivers/joint/joint_surveys/options.py

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,7 @@
1414
from pathlib import Path
1515
from typing import ClassVar
1616

17-
from geoapps_utils.utils.importing import GeoAppsError
18-
from pydantic import model_validator
17+
from pydantic import field_validator, model_validator
1918

2019
from simpeg_drivers import assets_path
2120
from simpeg_drivers.joint.options import BaseJointOptions
@@ -35,6 +34,19 @@ class JointSurveysOptions(BaseJointOptions):
3534

3635
models: ConductivityModelOptions
3736

37+
@field_validator("group_a", "group_b", "group_c")
38+
@classmethod
39+
def no_mvi_groups(cls, val):
40+
if val is None:
41+
return val
42+
43+
if "magnetic vector" in val.options.get("inversion_type", ""):
44+
raise ValueError(
45+
f"Joint inversion doesn't currently support MVI data as passed in "
46+
f"the group: {val.name}."
47+
)
48+
return val
49+
3850
@model_validator(mode="after")
3951
def all_groups_same_physical_property(self):
4052
physical_properties = [k.options["physical_property"] for k in self.groups]

0 commit comments

Comments
 (0)