Deltares / Deltares/imod-python
[FEATURE] - HFB support for iMOD5 models with linearrings
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 41
- Forks
- 12
- Avg merge
- 21h 8m
- Merged PRs (30d)
- 1
Description
@FransRoelofsen ran into this for MORIA and after discussion with @Huite:
iMOD5 supported providing HFBs as linearrings as well in the GEN file. There are models which rely on GEN files containing both linestrings as wells as linearrings in a single file. We now automatically convert linearrings in GEN files to polygons. I think this is what most users expect. In addition, MODFLOW6 HFBs now only expect linestrings. Sometimes users may want to provide a linearring which could be a building pit or something. I think this should be a linearring, as polygons would lead to headaches: What to do with polygons with interior rings?
We should therefore do the following to support these:
- In GEN file reader: Split up linestrings and polygons in different lists, separately convert to shapely objects, then merge together into single GeoDataframe
- in iMOD5 HFB converter: Convert polygon to linearring
- in MODFLOW6 : Add support for providing HFBs as linearrings, to simulate things like building pits.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the GEN file reader, the iMOD5 HFB converter, and MODFLOW6 HFB handling described in the issue. Trace how linestrings, polygons, and linearrings are currently converted, then verify that mixed GEN geometries are preserved, iMOD5 polygons become linearrings, and MODFLOW6 accepts linearrings for building pits.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- data
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100