Skip to content

Conversation

@jsignell
Copy link
Collaborator

@jsignell jsignell commented Sep 4, 2025

Builds off of #42 but with a newer base image so that the zarr and xarray are new enough to support virtualizarr >= 2.0.0

@jsignell jsignell force-pushed the pangeo-2025.08.14-vz2-ic1 branch from 4a11b1b to 99df022 Compare September 4, 2025 17:24
@jsignell
Copy link
Collaborator Author

jsignell commented Sep 4, 2025

We can try out this image in vedahub by using a custom image pointing to public.ecr.aws/nasa-veda/pangeo-notebook-veda-image:99df0220fb79

@jsignell
Copy link
Collaborator Author

jsignell commented Sep 8, 2025

Ok now I pinned geoviews slightly higher and ran all the notebooks

@jsignell
Copy link
Collaborator Author

jsignell commented Sep 8, 2025

New image at public.ecr.aws/nasa-veda/pangeo-notebook-veda-image:b529f65 @maxrjones

@wildintellect
Copy link
Contributor

🤔 the files look good to me, including the test, the image url you posted failed to load for me though. Also do we need @maxrjones to sign off on this since ODD requested these libraries?

@wildintellect
Copy link
Contributor

wildintellect commented Sep 9, 2025

Ah works if you use public.ecr.aws/nasa-veda/pangeo-notebook-veda-image:b529f655bc33 (longer hash code)

I also tested the Zarr Workshop 01 notebook works with this image.

(notebook) jovyan@jupyter-wildintellect:~$ mamba list zarr
List of packages in environment: "/srv/conda/envs/notebook"

  Name          Version  Build         Channel    
────────────────────────────────────────────────────
  virtualizarr  2.1.2    pyhd8ed1ab_0  conda-forge
  zarr          3.1.1    pyhe01879c_0  conda-forge

@jsignell
Copy link
Collaborator Author

Ooops sorry about that @wildintellect and thanks for running the workshop notebook as an additional test. I added a few cells to the test notebook that exercise very similar functionality.

@jsignell
Copy link
Collaborator Author

I'm going to go ahead and merge this since I think it's all good.

@jsignell jsignell merged commit 6d67aae into main Sep 10, 2025
1 check passed
@jsignell jsignell deleted the pangeo-2025.08.14-vz2-ic1 branch September 10, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants