There was some confusion (also by me) about the use of the LayeredHorizontalFlowBarrier classes, so @HendrikKok explained me they were intended for single layers. I therefore changed the following to clarify:
Rename to SingleLayerHorizontalFlowBarrier classes
Throw ValidationError if multiple layers are added to dataset
Added missing test for SingleLayerHorizontalFlowBarrier class
This means multiple SingleLayerHorizontalFlowBarrier need to be added to GroundwaterFlowModel object, whereas MODFLOW 6 only accepts one single HFB. So these have to be merged together. I think the best opportunity for this is to merge the low level Mf6HorizontalFlowBarrier objects.
Checklist
[x] Links to correct issue
[x] Update changelog, if changes affect users
[x] PR title starts with Issue #nr, e.g. Issue #737
Fixes #598
Description
There was some confusion (also by me) about the use of the
LayeredHorizontalFlowBarrier
classes, so @HendrikKok explained me they were intended for single layers. I therefore changed the following to clarify:SingleLayerHorizontalFlowBarrier
classesValidationError
if multiple layers are added to datasetThis means multiple SingleLayerHorizontalFlowBarrier need to be added to GroundwaterFlowModel object, whereas MODFLOW 6 only accepts one single HFB. So these have to be merged together. I think the best opportunity for this is to merge the low level
Mf6HorizontalFlowBarrier
objects.Checklist
Issue #nr
, e.g.Issue #737