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

DRAFT -- updates for recent changes to hydrofabric, etc. #8

Open
jameshalgren opened this issue Dec 22, 2023 · 1 comment
Open

DRAFT -- updates for recent changes to hydrofabric, etc. #8

jameshalgren opened this issue Dec 22, 2023 · 1 comment

Comments

@jameshalgren
Copy link

jameshalgren commented Dec 22, 2023

We made some temporary fixes in order to make the notebooks produce ngen input after some updates to ngen, the input datasets, and other dependencies. (These notebooks are amazing -- thank you @igarousi! and @Castronova!) These are things others are also working on, so these may quickly be obsolete notes. Commits and possibly a PR are forthcoming.

For the ngen-hydrofabric-subset:

As temporary fixes:

Bypassed the map by building a simple dataframe from one of the divide objects.
Updated the URL of the geopackage to the lynker-spatial s3 location
In subset.py, build the parquet name for the model_attributes from the new path in Lynker-spatial.com

Need to do

Adjust ngen.yaml template
Rebuild to use hf_subset altogether

For the ngen-create-cfe-forcings

As temporary fix:

Shunt away from Thredds and instead use direct connection to AORC zarr.
Need to figure out LQFRAC

Need to do

Transition to TEEHR or Ngen-Datastream method of forcing subsetting for better performance

@jameshalgren
Copy link
Author

@JoshCu

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

No branches or pull requests

1 participant