-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Labels
refactorRequires refactoringRequires refactoring
Milestone
Description
After speaking with @Manangka about #1744
The logic required for the clip_box method is now spread over several utilities. There is even some included in the imod/mf6/model.py, where it shouldn't be. Furthermore the data handling is convoluted now and it should be clarified in some way. I remember while working on #1744 that it took me a bit of time to find where and when processing steps took place and what was happening, which isn't a good sign.
The logic required is complex regardless, as there are quite some edge cases to handle, so we shouldn't expect such clarity that a new dev would understand every step at first glance, but some improvement is required.
What would help:
- Move functions in
imod/mf6/model.pyto clearer namespace - Add description of algorithm to clarify inner workings
- Refactor clip_box logic, possibly add some objects for internal use
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
refactorRequires refactoringRequires refactoring
Type
Projects
Status
📯 New