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

Sample Graphic Notebook #90

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Sample Graphic Notebook #90

wants to merge 10 commits into from

Conversation

michaelavs
Copy link

@michaelavs michaelavs commented Jul 13, 2021

@dopplershift
wrfout file is in this PR and environment has been updated to include wrf-python and netcdf4 dependencies. I think the majority of my issues are coming from imshow and incorporating a png file with cartopy projections.

@github-actions
Copy link

🚀 📚 Preview for git commit SHA: 140af86 at: https://61040e5a2dce1971e60f4b4f--pythia-foundations.netlify.app

@brian-rose
Copy link
Member

Hi @michaelavs, if you merge in the latest updates from main branch, it should fix the build failures here.

@brian-rose brian-rose added the hackathon Issue highlighted for active hackathon session label Aug 5, 2021
@michaelavs michaelavs requested a review from a team as a code owner September 14, 2021 19:30
@michaelavs michaelavs requested review from dcamron and r-ford and removed request for a team September 14, 2021 19:30
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@michaelavs michaelavs requested a review from a team as a code owner September 14, 2021 19:33
@michaelavs michaelavs requested review from dopplershift and removed request for a team September 14, 2021 19:33
@github-actions
Copy link

🚀 📚 Preview for git commit SHA: 88a42aa at: https://6140fbae64155600bb7af9fd--pythia-foundations.netlify.app

@michaelavs
Copy link
Author

I wanted some input on the file used in the python file to create the gif. The subsetted version of it is 409MB which is over the size that GitHub recommends/allows. I'm not sure if the code to create the gif is going to be built or not on the foundations website, if it isn't (either because it will be an inactive code or just not included), then there isn't really an issue. But if the code will be built/run, then I'd like to make sure that there won't be issues with the data file being the size it is.

Also, definitely looking for feedback on this option for the graphic. It's the third rendition of a graphic for the front page and is a bit different than what was discussed in the other PR/issue for creating the graphic. This dataset shows precipitation data for the timeframe of the Boulder floods (with a few additional days to add time to the gif loop). I think it helps to showcase a fun thing that can be done with Python for data visualization and goes with the goal of Project Pythia with respect to educating scientists on how to use Python.

@dopplershift
Copy link
Contributor

@michaelavs Is there somewhere I can get to that file? I'd love to see if there's anything else to be done to try to reduce the data size.

@michaelavs
Copy link
Author

michaelavs commented Sep 16, 2021

@dopplershift
The data set can be found here: https://daac.ornl.gov/cgi-bin/dsviewer.pl?ds_id=1328
Then scroll down a little bit to "Data Files", add the filter of "prcp_2013", and the original dataset is "daymet_v3_prcp_2013_na.nc4" (1.1GB).
I believe you will need an EarthData account to download it.

@github-actions
Copy link

github-actions bot commented Oct 28, 2021

This pull request is being automatically built with GitHub Actions and Netlify. To see the status of your deployment, click below.

🔍 Git commit SHA: 07c543e
✅ Deployment Preview URL: https://617ac59e80d8210ce5b87fe4--pythia-foundations.netlify.app

@dopplershift
Copy link
Contributor

Ok, managed to get the dataset down to 99M with some more subsetting from THREDDS and local rechunking for better compression. I've submitted a PR to the pythia-datasets repo to add the file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hackathon Issue highlighted for active hackathon session
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants