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

Enhancement to not update GUNW if Weather Model is Not Available for SLC Dates #526

Closed
cmarshak opened this issue Apr 27, 2023 · 5 comments · Fixed by #529
Closed

Enhancement to not update GUNW if Weather Model is Not Available for SLC Dates #526

cmarshak opened this issue Apr 27, 2023 · 5 comments · Fixed by #529
Assignees
Labels
enhancement New feature or request

Comments

@cmarshak
Copy link
Collaborator

For the ARIA-GUNW workflow, we do not want the operator to necessarily have to worry about the temporal availability of a weather model because the new GUNW will have additional layers that we still want to deliver to the DAAC even if RAIDER cannot be applied.

Examples:

  1. Processing S1 data within 3 months of current date (no HRRR data will be available).
  2. Processing the Aleutian Volcano chain but we can only go back to 2018 with HRRR there.

Will access weather model's instance variable _valid_range here: https://github.com/dbekaert/RAiDER/blob/dev/tools/RAiDER/cli/raider.py#L496

Will be very straightforward.

@cmarshak cmarshak added the enhancement New feature or request label Apr 27, 2023
@cmarshak cmarshak self-assigned this Apr 27, 2023
@dbekaert
Copy link
Owner

@cmarshak so this enhancement is in the operator notebook or on the raider side?

@cmarshak
Copy link
Collaborator Author

cmarshak commented Apr 27, 2023

Would be in Raider - there would be a function that would return True / False within this entrypoint depending on temporal availability.

@dbekaert
Copy link
Owner

Ok, so you are asking for a raider function that depending on model returns validity period.

@cmarshak
Copy link
Collaborator Author

cmarshak commented Apr 27, 2023

I will write this in Raider - I am just putting it as an issue so that we have a clear understanding of what needs to be done and the problem this is solving. It's very straightforward. Can have Brett review so we are on the same page.

@cmarshak
Copy link
Collaborator Author

cmarshak commented Apr 27, 2023

Sorry - to your original question, it already exists in Raider.

It's just

weather_model = HRRR()
valid_begin_dt, valid_end_dt = weather_model._valid_range

Honestly, it (valid_range) should be a class variable - but that's for another time and place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants