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

update GFS and GEFS template #358

Merged
merged 4 commits into from
Aug 7, 2024
Merged

update GFS and GEFS template #358

merged 4 commits into from
Aug 7, 2024

Conversation

blaylockbk
Copy link
Owner

@blaylockbk blaylockbk commented Aug 6, 2024

  • Added if block for datetime to switch to GFS v16 file structure.
  • Added sfluxgrb product discovery (fixes Add 13km GFS product #333)
  • Fixed bug in gdas from azure
  • Updated the GEFS template to access earlier data.
  • added ncar_rda as source for GFS, except this source can only download full GFS files.

Question.

  • What is the difference between anl files and f000 files?

- Added if block for datetime to switch to GFS v16 file structure.
- Added sfluxgrb product discovery
- Fixed bug in gdas from azure
@blaylockbk blaylockbk added this to the Next Release milestone Aug 6, 2024
Copy link
Owner Author

@blaylockbk blaylockbk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still would like to include access to NCEI for GFS before 2021. Need to investigate the NCEI file structure to see if it is at all like what is currently avaialble.

@vwgeiser
Copy link

vwgeiser commented Aug 7, 2024

They had an option to ask a question about the RDA dataset through UCAR service desk. I inquired about the index files, I can leave updates here when I get them.

I noticed on line 29 there is the code self.date < datetime(2021, 3, 23) is this the correct implementation for a date check in Herbie? Could this be used in my current attempt at solving a similar date check problem in #355 ?

@blaylockbk
Copy link
Owner Author

Thanks @vwgeiser for your code comment in #355. It was very useful. I used that in e47ecba. There was a slight change in the way the lead time was expressed in the file name between dates (f## versus f###).

I only tested this a little, but it seems to work.

FYI: I appreciate your error catching in your code for if a user supplied a bad date, but for Herbie I prefer not raising an error here. Herbie returns None for H.grib when it didn't find a file in any of the URLs, and I leave it up to the user to handle that case and investigate why none of the URLs in H.SOURCES are not valid. My reasoning is that the range of dates the different cloud platforms provide data for change over time.

@blaylockbk blaylockbk changed the title update GFS template update GFS and GEFS template Aug 7, 2024
@vwgeiser
Copy link

vwgeiser commented Aug 7, 2024

@blaylockbk Gotcha, makes sense about the returning None vs throwing an error. I appreciate the mention, I'll be testing this today with my workflow and I'll let you know if anything else pops up.

@vwgeiser
Copy link

vwgeiser commented Aug 7, 2024

@blaylockbk These changes look good. I updated my local Herbie installation and it works in addressing some of the issues in #325 and #355. Bummer about the no index files for ncar_rda, I'll let you know if I can think of any other work around.

Copy link

@vwgeiser vwgeiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good. Addresses issues in #325 and #355.

@blaylockbk blaylockbk merged commit 047ea49 into main Aug 7, 2024
8 checks passed
@blaylockbk blaylockbk deleted the revamp-gfs-template branch August 7, 2024 21:16
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

Successfully merging this pull request may close these issues.

Add 13km GFS product
2 participants