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

Upgrade: Pangeo version bump to 2024-11-11 #31

Merged
merged 1 commit into from
Jan 25, 2025
Merged

Conversation

wildintellect
Copy link
Contributor

@wildintellect wildintellect commented Jan 8, 2025

Addressing #30
I've tested loading in https://staging.hub.openveda.cloud using public.ecr.aws/nasa-veda/pangeo-notebook-veda-image:c76b967c7648

Any particular things to test once it's booted?

@wildintellect wildintellect added the enhancement New feature or request label Jan 8, 2025
@wildintellect wildintellect self-assigned this Jan 8, 2025
@sunu
Copy link
Member

sunu commented Jan 9, 2025

Any particular things to test once it's booted?

@wildintellect During previous upgrades, @jsignell mentioned running through the veda-docs quickstart notebooks as a sanity check on the VEDA staging hub.

@wildintellect
Copy link
Contributor Author

I tried a couple of them, @jsignell did we have a quick way to run all of them?

@wildintellect wildintellect marked this pull request as ready for review January 10, 2025 00:09
@jsignell
Copy link
Collaborator

Not really - I just open the jupyter hub and run them one by one using the "run all cells" button 🤷🏻 and see if I get any warnings or errors.

@sunu
Copy link
Member

sunu commented Jan 21, 2025

@wildintellect I'd be happy to help test the notebooks if you want. Once we verify there are no errors or warnings, I think we can proceed with merging the PR.

@wildintellect
Copy link
Contributor Author

@sunu I hit a bump when I realized the Ram requirements are mentioned in a collapsed section of each notebook. Make sure you pick a big enough instance. I'll see if anyone else has time to test too.

@jsignell
Copy link
Collaborator

jsignell commented Jan 22, 2025

I realized the Ram requirements are mentioned in a collapsed section of each notebook. Make sure you pick a big enough instance

I think if you just pick 16GB you will be good. Also just to be clear it is totally fine to run all the notebooks in one instance. Just run them one at a time

@wildintellect
Copy link
Contributor Author

Ticket to track testing NASA-IMPACT/veda-docs#200

@wildintellect
Copy link
Contributor Author

@sunu I'm done testing all the notebooks, this change is not the cause of any failures (as noted in the linked ticket)

@sunu sunu merged commit e4d50b7 into main Jan 25, 2025
2 checks passed
@sunu
Copy link
Member

sunu commented Jan 25, 2025

Thanks @wildintellect! I've merged the PR and created a new tag 2024.11.11-v1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants