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

r.sim.water: add info to manual, add info about progress #4662

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

metzm
Copy link
Contributor

@metzm metzm commented Nov 6, 2024

  • r.sim.water is a module with a number of important but not well documented parameters. This PR adds more information about some parameters to the manual.
  • It takes quite some time to run a simulation, another message is added to the progress of the simulation.

This PR tries to add info requested in #4475 to the manual

@metzm metzm added manual Documentation related issues raster Related to raster data processing C Related code is in C labels Nov 6, 2024
@metzm metzm added this to the 8.5.0 milestone Nov 6, 2024
@metzm metzm requested a review from petrasovaa November 6, 2024 21:39
@github-actions github-actions bot added HTML Related code is in HTML module docs labels Nov 6, 2024
@metzm metzm requested a review from petrasovaa November 7, 2024 21:25
@hmitaso
Copy link
Contributor

hmitaso commented Nov 9, 2024

This sentence may be confusing "The model is for shallow water flow, where "shallow" is defined by hmax"- Perhaps the paragraph could start with "the model tries to keep water "shallow" with maximum shallow water depth defined by hmax value (default is 0.3m). If hmax is reached then .... here goes the text that explains the increase in diffusion rate and related parameters .... The rest of the text that explains when these parameters may need to be adjusted sounds good to me.

@metzm
Copy link
Contributor Author

metzm commented Nov 13, 2024

This new description comes from #4475, maybe @petrasovaa can clarify? I am not sure if the model tries to keep water "shallow". After hmax is reached, water flow can more easily overcome obstacles like sinks. "shallow" might here refer not to water depth, but to noise or sinks in the DEM used as input.

@metzm
Copy link
Contributor Author

metzm commented Nov 14, 2024

@hmitaso New try to explain diffusion in order to keep water shallow and to overcome obstacles in ad725c7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Related code is in C docs HTML Related code is in HTML manual Documentation related issues module raster Related to raster data processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants