Tutorials for harmony-py webinar short - #109
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
The test that is failing is hinting that, for Linux, not all of the libraries are present in the environment for the notebook. The notebook does work for my mac so the tutorial won't be impacted. My hope is to merge this PR so the notebook can be shared, then come back to address this issue. |
|
Looks good to me! |
| @@ -0,0 +1,880 @@ | |||
| { | |||
There was a problem hiding this comment.
For final suggest, suggest active voice. "Harmony is a data reduction service that allows..."
Suggest "Not all data collections/sets have transformation services and not all transformation services are available for every dataset"
harmony-py is a "wrapper" for Harmony's RESTful API
Reply via ReviewNB
| @@ -0,0 +1,880 @@ | |||
| { | |||
There was a problem hiding this comment.
| @@ -0,0 +1,880 @@ | |||
| { | |||
There was a problem hiding this comment.
| @@ -0,0 +1,880 @@ | |||
| { | |||
There was a problem hiding this comment.
I would explain why we have to do these steps. Something along the lines of...
"Once we submit the request, Harmony processes the request to transform and subset the data. This takes time. The job_id is used to track the status of the processing and also to return links to the transformed data."
Reply via ReviewNB
| @@ -0,0 +1,880 @@ | |||
| { | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
results? I thought futures was weird as well, but results can get messy with the function.
3 participants
There are two notebooks (one rendered, one not) for adding to the NASA webinar shorts folder. These are fully typed out, but a working one for the webinar will have a blank code cell for the harmony request as well as other interactive lines.