Skip to content

Split compression type/quality into separate section, add JPEG info#137

Merged
melissalinkert merged 2 commits intoglencoesoftware:masterfrom
melissalinkert:quality-documentation
Sep 11, 2025
Merged

Split compression type/quality into separate section, add JPEG info#137
melissalinkert merged 2 commits intoglencoesoftware:masterfrom
melissalinkert:quality-documentation

Conversation

@melissalinkert
Copy link
Copy Markdown
Member

I debated linking https://bio-formats.readthedocs.io/en/latest/users/comlinetools/conversion.html#cmdoption-bfconvert-quality, but opted not to for the moment, particularly since the other options on that page might cause confusion.

This also adds a note about using --rgb with brightfield data, and a note about why LZW is the default compression.

Copy link
Copy Markdown
Member

@sbesson sbesson left a comment

Choose a reason for hiding this comment

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

Thanks for the addition. Only question is to find whether the TIFF reference index that had been nicely maintained by Joris Van Damme over the years has found a new home

Comment thread README.md Outdated
The input tile directory must contain a full pyramid in a Zarr container.

If the `--rgb` option is set, the data will be stored in the OME-TIFF using
the [chunky format](https://www.awaresystems.be/imaging/tiff/tifftags/planarconfiguration.html)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sadly, awaresystems.be/imaging/tiff is now 404. Looks like this reference index is no longer maintainer - see this thread https://www.mail-archive.com/tiff@lists.osgeo.org/msg00421.html and I don't know if the content has been officially migrated.
For this repository we probably either need to identify a new reference page or link to the TIFF specification directly

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

93d8ebb fixes this as we did in Bio-Formats documentation (see ome/bio-formats-documentation#387), and adds a link to the specification. Definitely open to better suggestions, but unfortunately I don't know of a reliable alternative reference page.

@sbesson
Copy link
Copy Markdown
Member

sbesson commented Sep 11, 2025

@mabruce @emilroz anything else you would like to be added to this section?

@emilroz
Copy link
Copy Markdown
Member

emilroz commented Sep 11, 2025

All looks good to me.

@melissalinkert
Copy link
Copy Markdown
Member Author

Merging so we can release 0.8.0.

@melissalinkert melissalinkert merged commit f62fff0 into glencoesoftware:master Sep 11, 2025
4 checks passed
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