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 MZ example notebooks #62

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

m-zoerner
Copy link

New example workflows. They contain image processing for radar reflectivity.

@github-actions
Copy link

github-actions bot commented Aug 5, 2022

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 19eb65c
✅ Deployment Preview URL: In Progress

@brian-rose
Copy link
Member

Hey @mgrover1 just wondering if you can take a look at this? If you're too busy, feel free to tag other Pythians!

Copy link
Contributor

@mgrover1 mgrover1 left a comment

Choose a reason for hiding this comment

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

The main issues right now are:

  • Access to the data (Cells reading in the raw NEXRAD data, plot to grid, and save)
  • Adhering to the notebook template, with the summary, table of contents, and references
  • Ensuring there is a small enough subset to execute in a reasonable amount of time

I appreciate you submitting this PR @m-zoerner - this is something we can work together on (or I can move forward on with your feedback).

@m-zoerner
Copy link
Author

Hey Max, sorry for the massive delay getting back to you on this. Now that I'm back at school things have been super busy. I can work on picking out a smaller subset of NEXRAD data to make accessible for this example as well as add the summary, TOC, and references.

@mgrover1
Copy link
Contributor

mgrover1 commented Oct 3, 2022

No worries @m-zoerner ! That sounds like a great plan :) let me know if you need any help moving this along!

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.

3 participants