Skip to content

Make logging of unknown fields in OME metadata less severe#75

Open
stefanhahmann wants to merge 1 commit intozarr-developers:mainfrom
stefanhahmann:stefanhahmann-patch-1
Open

Make logging of unknown fields in OME metadata less severe#75
stefanhahmann wants to merge 1 commit intozarr-developers:mainfrom
stefanhahmann:stefanhahmann-patch-1

Conversation

@stefanhahmann
Copy link
Copy Markdown
Contributor

This warning fired e.g. when reading https://livingobjects.ebi.ac.uk/idr/zarr/v0.5/idr0033A/BR00109990_C2.zarr/0 Since, this log can actually be ignored, I propose to lower the severity of the log.

grafik

This warning fired e.g. when reading https://livingobjects.ebi.ac.uk/idr/zarr/v0.5/idr0033A/BR00109990_C2.zarr/0
Since, this log can actually be ignored, I propose to lower the severity of the log.
@stefanhahmann stefanhahmann marked this pull request as ready for review May 7, 2026 11:46
@normanrz
Copy link
Copy Markdown
Member

normanrz commented May 8, 2026

Hmm. Why is it a problem that this is a warning?
I am not sure it is a solved question whether it is permissible to have additional keys in the OME-Zarr metadata. I would lean towards notifying users about such additional metadata because the additional keys could have a semantic meaning.

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