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

updated radiosondes to v3 #72

Merged
merged 1 commit into from
Jul 27, 2021
Merged

updated radiosondes to v3 #72

merged 1 commit into from
Jul 27, 2021

Conversation

d70-t
Copy link
Contributor

@d70-t d70-t commented Jul 27, 2021

This includes a change from denby.io to Aeris.

see #26 for some discussion about this

This includes a change from denby.io to Aeris.

see eurec4a#26 for some discussion about this
@d70-t d70-t added data source changed Fix for vanished source locations. data version update labels Jul 27, 2021
@d70-t d70-t requested a review from RobertPincus July 27, 2021 13:09
@d70-t
Copy link
Contributor Author

d70-t commented Jul 27, 2021

I think this is the minimal change required for the radiosondes. The issue mentioning the update (#26) is definitely overdue and currently the old version also is not available.

Probably we'll want to create more entries for other processing levels and may want to put the radiosondes at a different place in our hierarchy (the current location doesn't match our platform/product scheme too well). But those changes seem to be independent of this fixup / version upgrade.

Copy link
Contributor

@RobertPincus RobertPincus left a comment

Choose a reason for hiding this comment

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

Simple and clean. I like the move to AERIS too. Agreed that we might revisit the radiosondes-in-catalog question.

@d70-t d70-t merged commit 94710ee into eurec4a:master Jul 27, 2021
@d70-t d70-t deleted the radiosondes_v3 branch July 27, 2021 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data source changed Fix for vanished source locations. data version update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants