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

Icechunk docs update #627

Open
2 of 3 tasks
abarciauskas-bgse opened this issue Jan 25, 2025 · 0 comments
Open
2 of 3 tasks

Icechunk docs update #627

abarciauskas-bgse opened this issue Jan 25, 2025 · 0 comments

Comments

@abarciauskas-bgse
Copy link
Contributor

abarciauskas-bgse commented Jan 25, 2025

@paraseba has already done a lot of great work updating docs. I am trying to help out with a few things I noticed still outstanding so opening this issue to track those docs issues I found.

  • https://icechunk.io/icechunk-python/virtual/ - I don't think this example will run using the install commands in the issue Virtual Dataset Workflow Tracking Issue #197, specifically because if you install icechunk separately from VirtualiZarr, it will install VirtualiZarr 1.2.0 and icechunk 0.1.0a15, which are incompatible. I tried to use specific previous versions of icechunk with VirtualiZarr but am constantly running into issues. I could continue to try and resolve which Virtualizarr / previous icechunk version was stable, perhaps it was before 1.2.0. OR we could just remove the existing content from that page and say a future version of VirtualiZarr will be released that is stable with icechunk. Open to other suggestions! cc @mpiannucci
    a. Examples: Array codecs being out of order (I think a result of zarray._v3_codec_pipeline() in VirtualiZarr) or TypeError: AsyncGroup.create_array() got an unexpected keyword argument 'overwrite' (probably some zarr python incompatibility with the VirtualiZarr version) or, in icechunk 0.1.0a8 I found session.store returned TypeError: Can't instantiate abstract class IcechunkStore with abstract methods empty, with_mode (using a local filesystem store).

  • https://icechunk.io/sample-datasets/ - should this page be removed or can the s3://earthmover-sample-data/icechunk/oisst.2020-2024/ store be updated? I believe it won't work with the latest icechunk version

  • I haven't tried this one out yet but I think https://icechunk.io/icechunk-python/notebooks/demo-s3/ needs to be updated as its still using StorageConfig.

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

No branches or pull requests

1 participant