nens / threedi-api-qgis-client

3Di Models & Simulations plugin
https://plugins.lizard.net/
Other
2 stars 0 forks source link

Simulation wizard: Add substance concentrations to boundary conditions page #536

Closed leendertvanwolfswinkel closed 3 weeks ago

leendertvanwolfswinkel commented 1 month ago

@hoanphungt I tested it and works well when I upload both the boundary condition time series itself and the substance concentration time series. But it does not work when I use the boundary conditions from the template and add the substance concentrations to them. How diffifcult would it be to also make that work? If it is to difficult, we could make a separate ticket and do that later. But then we need to disable the concentrations upload widgets if no CSV has been chosen for the boundary conditions yet (separate logic for 1D and 2D of course)

hoanphungt commented 1 month ago

@leendertvanwolfswinkel Can you test again using template boundary conditions? I made it work and pushed a new commit to the connected PR.

leendertvanwolfswinkel commented 4 weeks ago

@hoanphungt tested, please merge Also works with boundary conditions from template if I convert the time units in the substance concentration time series to seconds myself