-
Notifications
You must be signed in to change notification settings - Fork 4
WIP: data in the cloud module #44
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the idea of having dependencies specified for each module, but I think logistically, it makes the most sense to toss these in to one big conda environment users will use for the whole workshop.
|
A couple notes as I'm reading:
I did some tweaks that I felt confident were uncontroversial, and I'm going to merge what we have! |
The prompt "List the files available following at this prefix on AWS S3 storage" left me expecting the list of files to be output. If we intend to not output it we could instead say "Create a list of the files..."?
|
Do you want this notebook to show up as executed in the course materials website? Or do you want the participants to execute it themselves to see the results? |
I apologize, I didn't read this carefully :D I'll make it a draft PR. |
Not quite ready for a review yet, will resume work on this ~Dec 1
🔍 Preview: https://geojupyter-workshop-open-source-geospatial--44.org.readthedocs.build/
Note: This Pull Request preview is provided by ReadTheDocs. Our production website, however, is currently deployed with GitHub Pages.