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

Option to crop uncertainty intervals at zero for RC and Q time series? #83

Open
benRenard opened this issue Sep 13, 2024 · 1 comment
Open
Labels
enhancement New feature or request question Further information is requested

Comments

@benRenard
Copy link
Member

BaRatinAGE generates total uncertainty intervals that can go negative, due to the structural component. Several users have suggested that such intervals should be cropped at zero. I believe this should not be the default behavior, because this simply results from BaRatin assumptions, and I'd rather show the direct result of these assumptions. But we could provide an option to do that if the user decides so. Questions:

  1. Should we do it, or rather let the user take care of that by post-processing BaRatinAGE exports?
  2. If we implement the option, should it be a general option that always remains activated, or an option that should be activated for each case study?

@IvanHeriver, @JeromeLeCoz, what do you think?

@benRenard benRenard added enhancement New feature or request question Further information is requested labels Sep 13, 2024
@JeromeLeCoz
Copy link
Collaborator

  1. I'm not sure, but it may be useful at stations with precise controls for discharges going to zero. Not the default behavior anyway, agreed. If it's easy to implement, why not (that would save arguments with users)
  2. For each case study, I'd say. As you said earlier, I also believe it is a useful warning when it happens (except for very small flows).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants