Martijn Visser
Martijn Visser
Yes we will remove both `edge_id` (https://github.com/Deltares/Ribasim/issues/1310) and the "vertical" flows from flow.arrow, either would fix this.
Not sure why, but this is fixed now:  Technically the greyed out ID column should read "edge_id" not "fid", since the name of the primary key column in the...
TOML support [local date time](https://toml.io/en/v1.0.0#local-date-time) and [offset date time](https://toml.io/en/v1.0.0#offset-date-time). Currently we use local. We could support offset date time for `starttime` and `endtime`, also as a way to indicate to...
In #2285 we are discussing making the FlowBoundary forward fill (block) interpolation by default, but making it configurable. This issue mentions the `UserDemand / time` which is still linearly interpolated....
These timeseries don't yet default to a block interpolation: - [ ] Concentration / time (? behavior not documented) - [ ] UserDemand / time - [ ] Pump /...
And addition from @SouthEndMusic that is in scope of this issue: > @Fati-Mon mentioned that from the documentation it is not clear how to set up a model in Ribasim...
Related to #208
The table already doesn't appear in any of our test models. In the add API it becomes obvious. ```python model.terminal.add(Node(14, Point(3.0, -2.0))) model.terminal.add(Node(14, Point(3.0, -2.0)), [terminal.Static()]) ```
Great, I updated the issue to just cover the Node case.
Let's throw an error on `edge.add`. Check for duplicates in a similar way to how #1717 checks for duplicate node ID adds; keep a set with the unique (from_node_id, to_node_id)...