-
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.
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 think this makes sense for the workshop. I also would like to provide the pixi files as a separate option, so that people interested in following along with the entire course could use the docker file but anyone who wants to try out just this module can use the pixi environment.
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 think that makes sense. Perhaps we should all do this for our modules. Any thoughts on keeping all these environment files up-to-date?
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'll plan to run the tutorial using either option to make sure everything runs correctly. I don't think we should try to get the environments to be identical, since the dependency solving will change between the more limited and larger environment sizes.
|
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.