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

Added potential recipe to destagger AUS2200 U and V wind components #22

Merged

Conversation

andrewbrown31
Copy link
Collaborator

This recipe again doesn't use the intake catalogue, as I'm still not sure where this is for AUS2200 data or if it exists at the moment (noting that the data is currently moving over to a different project as part of a publication process). Can update when I find the catalogue.

@navidcy I wanted to request @claire-gh to review but they must not be on the right list of users?

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Collaborator

@youstormorg youstormorg left a comment

Choose a reason for hiding this comment

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

Hi Andrew, I'm not too familiar with the reviewing process here, but this looks great! I'll try it out on AUS2200 data. I approved it and did a "squash and merge" but not sure how to get this onto the cookbook page.

@youstormorg youstormorg merged commit f586432 into ACCESS-Community-Hub:main Dec 13, 2024
2 checks passed
@navidcy
Copy link
Contributor

navidcy commented Dec 13, 2024

@youstormorg thanks for reviewing this!!

Small minor note: the GitHub etiquette is to leave the merge to be done by the person who opened the PR (in case they had something else to add).

@youstormorg
Copy link
Collaborator

@youstormorg thanks for reviewing this!!

Small minor note: the GitHub etiquette is to leave the merge to be done by the person who opened the PR (in case they had something else to add).

Thank you! I'll remember this next time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants