Data access tutorial #11
Conversation
|
Adding the Data Access chapter to the SnowEx website-2024 Jupyter Book. |
ref: https://earthaccess.readthedocs.io/en/latest/howto/authenticate/#authenticate-with-earthdata-login I just added them @mikala-nsidc! So the CI workflows will have access via a bot "uwhackweek" account |
conda/README.md
Outdated
There was a problem hiding this comment.
You can remove the conda/ folder from this PR (I'll do a separate update of the CI environment to match cryocloud in #12)
There was a problem hiding this comment.
thanks @scottyhq ! Just to clarify - should I revert my commit that contains the change requests to anything under conda/? Or do you mean actually remove the Conda/ folder from my fork and push that change? Thanks for any help!
There was a problem hiding this comment.
I meant revert your change or copy from the new main branch. I think there are different ways to do this. Maybe this?:
git checkout main
git pull upstream main
gh pr checkout 11
git merge main
git checkout HEAD -- conda/
git push
|
@scottyhq I reverted the commit I made yesterday that requested changes to files in conda/. Let me know if this works, or if I need to do something else to make the pull request mergeable. Thanks! |
| <figure> | ||
| <center> | ||
| <img width="80%" src='./images/DAAC_map_with_ECS.jpg'/> | ||
| </center> | ||
| </figure> |
There was a problem hiding this comment.
This syntax is not rendering on CryoCloud, could you please try this syntax https://mystmd.org/guide/figures#image-directive
|
After #13 if you merge the main branch again I think your notebooks will pick up the EARTHDATA credentials and successfully execute |
| "source": [ | ||
| "downloaded_files = earthaccess.download(\n", | ||
| " results,\n", | ||
| " local_path='./Downloads',\n", |
There was a problem hiding this comment.
Consider changing the download path to /tmp for tutorial data (https://docs.2i2c.org/user/topics/data/filesystem/#the-tmp-directory)
|
@mikala-nsidc This is looking great. I would love to see your and Gail's hard work putting this content together recognized if you're comfortable adding your names to the notebooks and/or overview page. |
|
I forgot to add in my commit message that I also changed the format of the figures, so hopefully they will render in the book in cryocloud now. Let me know, if not! |
mikala-nsidc
force-pushed
the
data-access-tutorial
branch
from
0e23eb8 to
84475f7
Compare
|
That last force push was to overwrite the commits by egreckase and go back to where we were prior to those. |
mikala-nsidc
force-pushed
the
data-access-tutorial
branch
from
cb57bef to
a480e72
Compare
|
I see those commits @scottyhq ! Thanks for adding those for me. |
the website build through GitHub actions is failing, because the notebooks need to authenticate with Earthdata Login. We're doing that authentication with earthaccess and so we could provide credentials to the notebook via the following environment variables
EARTHDATA_USERNAME, EARTHDATA_PASSWORD
could you set those as repository secrets? do you already have a shared account for this?