Skip to content

Handle case-insensitive platforms in Zarr hierarchy#291

Merged
ziw-liu merged 6 commits into
mainfrom
warn-key-case
Apr 4, 2025
Merged

Handle case-insensitive platforms in Zarr hierarchy#291
ziw-liu merged 6 commits into
mainfrom
warn-key-case

Conversation

@ziw-liu
Copy link
Copy Markdown
Contributor

@ziw-liu ziw-liu commented Apr 2, 2025

Fix #253.

Case coercing in key lookup only occurs when the platform requires it (i.e. not happening on Linux anymore).

@ziw-liu ziw-liu added this to the 0.2.0 milestone Apr 2, 2025
@ziw-liu ziw-liu changed the title Handle case-insensitive platforms Handle case-insensitive platforms in Zarr hierarchy Apr 2, 2025
@ziw-liu ziw-liu marked this pull request as ready for review April 2, 2025 23:01
Copy link
Copy Markdown
Contributor

@ieivanov ieivanov left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@ziw-liu ziw-liu merged commit dc704aa into main Apr 4, 2025
7 checks passed
@ziw-liu ziw-liu deleted the warn-key-case branch April 4, 2025 22:06
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.

create_position fails when inconsistent row/column capitalization is used

2 participants