Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

setup_rivers does not update river width and depth #283

Closed
2 tasks done
shartgring opened this issue Jun 26, 2024 · 3 comments
Closed
2 tasks done

setup_rivers does not update river width and depth #283

shartgring opened this issue Jun 26, 2024 · 3 comments
Assignees
Labels
bug Something isn't working setup methods Issue related to existing or new setup methods workflows Issue linked to the workflows

Comments

@shartgring
Copy link
Collaborator

shartgring commented Jun 26, 2024

HydroMT-Wflow version checks

  • I have checked that this issue has not already been reported.
  • I have checked that this bug exists on the latest version of HydroMT-Wflow.

Reproducible Example

Using setup_rivers() to update a model in a Python script does not update all parameter maps. For example, updating river maps with upa=100 to upa=5, resulting in a larger river map, but the wflow_riverwidth and RiverDepth map still only has width values for the cells created by upa=100 and not upa=5

staticmaps.zip

Current behaviour

Does not update river width and depth when upa is changes

Desired behaviour

Update river width and depth for all new river cells, not old ones

Additional context

No response

@shartgring shartgring added bug Something isn't working needs refinement issue still needs refinement workflows Issue linked to the workflows setup methods Issue related to existing or new setup methods and removed needs refinement issue still needs refinement labels Jun 26, 2024
@shartgring shartgring changed the title setup_rivers does not update river width setup_rivers does not update river width and depth Jun 26, 2024
@shartgring shartgring self-assigned this Jun 26, 2024
@shartgring
Copy link
Collaborator Author

shartgring commented Jun 26, 2024

Looked into it and think I know what it causing it, will try to fix it myself this week. I think someway in which attribute columns in the gdf are checked and parts in the order/checking of the code are made to work with building models, not updating

@shartgring
Copy link
Collaborator Author

shartgring commented Jun 27, 2024

I found what was causing it, also partly an error (typo) from my side. When river_geom_fn is not specified but a model does already contain a river depth and width, the river depth and width maps are not added, so the old width and depth maps remain. What would be desired behaviour, to remove the old width and depth maps when setup_rivers is called.

Or add a width and depth map when river_geom_fn is not specified with the minimum depth and widths? I think this makes sense, and that map then automatically gets the same mask as the river, preventing that these is a mismatch between the number of river cells when a model is updated

@shartgring
Copy link
Collaborator Author

I will close the issue as it is already listed in #195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working setup methods Issue related to existing or new setup methods workflows Issue linked to the workflows
Projects
None yet
Development

No branches or pull requests

1 participant