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

Create dataset_to_xarray_notebook.ipynb #318

Closed
wants to merge 1 commit into from

Conversation

ThomasMGeo
Copy link

@ThomasMGeo ThomasMGeo commented Nov 17, 2022

Added a notebook to easily convert datasets to xarray using siphon. I could not get all of the interactivity that I wanted to extend this notebook into plotting.

Future notebooks will cover plotting and processing. Will think about appropriate places to put them (either here or project pythia, xarray docs, etc.).

Thanks!
-TM


This change is Reviewable

Added a notebook to easily convert datasets to xarray using siphon
@CLAassistant
Copy link

CLAassistant commented Nov 17, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@haileyajohnson
Copy link

Woo! 🎉
I have 2 questions:

  1. What type of dataset is this notebook targeting?
  2. Is this something that is useful to distribute with the TDS war, or something that we should just put on our server (in which case we'd want to transfer this PR over to TdsConfig)

@ThomasMGeo
Copy link
Author

ThomasMGeo commented Nov 17, 2022 via email

@haileyajohnson
Copy link

Ok great - I think I'd lean toward trying it out on our THREDDS server first, which would mean opening a PR on the TdsConfig repo. That's my bad for pointing you to the wrong issue at the start: this one
I can help with configuring it for those datasets if you'd like

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