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

Recalculate Wyoming #730

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tmharty
Copy link

@tmharty tmharty commented Sep 11, 2023

Description Of Changes

Allows for data from the Wyoming soundings site to be recalculated which overwrites a possibly incomplete cache on their site. I believe this is the issue pointed out in #310, but I am not entirely sure.

I am new to this process, so please forgive any mistakes I have made. This especially extends to the test that I included.

Checklist

This change allows for a user to request that the RAOB is
"Recalculated." This can be required if an earlier user of the
site requests the observation before the entire observation is
available. This results in an incomplete observation being cached.
Recalculation requires additional resources, so the default is to
not recalculate.
This is a partial test of recalc=True in WyomingUpperAir.request_data.
Since recalculation only has an appreciable effect based on the
unpredictable state of the Wyoming cache, this test only checks if
including the recal=True breaks the request.
@CLAassistant
Copy link

CLAassistant commented Sep 11, 2023

CLA assistant check
All committers have signed the CLA.

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.

Buffalo soundings cut off at 400 hpa for Sept 29 at 12 UTC through Siphon
2 participants