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

What does HistFlag do? #44

Open
calebbraun opened this issue May 29, 2019 · 1 comment
Open

What does HistFlag do? #44

calebbraun opened this issue May 29, 2019 · 1 comment
Assignees
Labels

Comments

@calebbraun
Copy link
Contributor

calebbraun commented May 29, 2019

It is unclear what the intent is of the HistFlag configuration variable. The wiki says

If True, channel storage and soil moisture files are saved; if False, channel storage and soil moisture files are loaded when using the GWAM runoff module.

This isn't exactly correct, as the MRTM routing module also uses this flag to check whether to load channel storage values.

The example configuration file has

# HistFlag = True, historic mode ; = False, future mode
HistFlag = True

but it isn't clear what is meant by historic mode or future mode.

This is also the only place where the truthy value is "True"/"False" (although "true" is allowed in some cases, and "t", "y", "yes", and "1" are even allowed in one case). The preferred alternative would be 0/1 like the other flag options.

@crvernon
Copy link
Member

The HistFlag is a relict from the version 1 setup. The intention was that the user could run the historical period and then save both channel storage and soil moisture files for which spin-up would have already been accounted for. These files then would replace the spin-up process for the GWAM runoff model and originally saved on runtime.

Then comes version 2 with alternate options for these models. Now we have models like the abcd model, now the default runoff model, that has its own spin-up protocol. So HistFlag became only useful with a GWAM configuration. Spin-up is now conducted off of the first n number of months that the user sets in the config; which is why we combine historical and future datasets in one run.

I can see the benefit for being able to initialize the abcd model with pre-spun data. Please explore the ability, and consequences, to do the following:

  • remove the old HistFlag and replace it with something more intuitive that could work with the abcd model as well
  • think about how this would be implemented with other core components that may be added as options with future development
  • build test as these test are being conducted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants