Closed ishita9 closed 1 year ago
Hello @ishita9! Thanks for updating this PR.
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Does anyone understand why travis is not running on this PR? @rcabell @ishita9
Does anyone understand why travis is not running on this PR? @rcabell @ishita9
Doesn't look like it's set up in the Integrations settings (only PEP8Speaks is listed currently). I'll look into it.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
wrfhydropy/core/domain.py | 1 | 2 | 50.0% | ||
<!-- | Total: | 1 | 2 | 50.0% | --> |
Totals | |
---|---|
Change from base Build 602: | -0.02% |
Covered Lines: | 3751 |
Relevant Lines: | 4271 |
In order to use USGS-USACE reservoir files, I first needed to update the hydro_namelist_patches.json file in the CONUS domain dir and added fields in the hydro_nlist:
Within wrfhydropy, in order for these files to be picked up by code, I needed to add the following two lines. This will add these files in the hydro_nlist: