-
Notifications
You must be signed in to change notification settings - Fork 9
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
s3 urls for community datastream #130
Comments
Anticipating retrospective output, planning on adding a For forcings generated from the operational NWM For forcings generated from the retrospective NWM |
The time axis for reanalysis (AnA) forcings moves backwards through time (tm00 > tm01). Should we store these files under a prefix like Something like this: https://ciroh-community-ngen-datastream.s3.amazonaws.com/reanalysis/ngen.20241029/VPU_01/ngen.t00z.analysis_assim_extend.forcing.tm00_tm27.conus.nc EDIT Flipping tm27 and t00 communicates that the time axis is moving forward in time. |
The forcings generated with v2.1 hydrofabric have had a prefix of "v2.1" added. Going forward, a hydrofabric prefix will be included for each file generated by datastream (v.2.2 now). |
What convention should the urls follow for datastream forcings and outputs in the ciroh community ngen datastream bucket?
Current ngen forcings url example: https://ciroh-community-ngen-datastream.s3.amazonaws.com/20241007/forcing_medium_range/forcings/VPU_01_forcings.nc
nwm convention for short range conus forcings files is
https://noaa-nwm-pds.s3.amazonaws.com/nwm.20241003/forcing_short_range/nwm.t00z.short_range.forcing.f001.conus.nc
...
https://noaa-nwm-pds.s3.amazonaws.com/nwm.20241003/forcing_short_range/nwm.t00z.short_range.forcing.f018.conus.nc
so ngen forcings created from the above nwm forcings (split by vpu) can be:
https://ciroh-community-ngen-datastream.s3.amazonaws.com/ngen.20241003/forcing_short_range/ngen.t00z.short_range.forcing.f001_f018.VPU_01.nc
The same pattern would be used for other time ranges like
medium_range
by swapping outshort_range
for the appropriate name.Thoughts? @jameshalgren @hellkite500 @ZacharyWills
The text was updated successfully, but these errors were encountered: