Skip to content

Conversation

@maxrjones
Copy link
Member

The kerchunk TIFF parser was never functional in VirtualiZarr (e.g., #291, #526), which prompted the development of virtual-tiff. Since the kerchunk tiff wrapper was never publicly documented or tested, I propose that we remove it from the code base.

  • Closes #xxxx
  • Tests added
  • Tests passing
  • Full type hint coverage
  • Changes are documented in docs/releases.rst
  • New functions/methods are listed in api.rst
  • New functionality has documentation

@TomNicholas
Copy link
Member

10000% Thanks for remembering about this. But the docs need to be scanned for references to remove, and replaced with links to virtual-tiff.

@maxrjones
Copy link
Member Author

maxrjones commented Jan 6, 2026

10000% Thanks for remembering about this. But the docs need to be scanned for references to remove, and replaced with links to virtual-tiff.

I used grep and claude to search for references, but none came up apart from historical references in the changelog. The table mentioning this parser was previously updated in #810.

Copy link
Member

@TomNicholas TomNicholas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks for checking.

@maxrjones maxrjones merged commit c67dcc1 into zarr-developers:main Jan 6, 2026
15 checks passed
@maxrjones maxrjones deleted the remove-kerchunk-tiff branch January 6, 2026 21:18
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.

2 participants