Skip to content

Add opt-in lossless TIFF compression for materialized outputs - #129

Merged
trissim merged 6 commits into
mainfrom
feat/compressed-artifact-output
Sep 24, 2026
Merged

trissim merged 6 commits into
mainfrom
feat/compressed-artifact-output

Conversation

@trissim

@trissim trissim commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Adds declaration-owned TiffConfig with NONE and DEFLATE choices and levels 1–9. The effective setting travels through ProcessingContext and applies only to disk TIFF writes, including named image and label artifacts and ordinary main-flow image materialization. Non-TIFF outputs retain existing writer kwargs; uncompressed remains the default.

PolyStore PR OpenHCSDev/PolyStore#11 is merged and published as v0.2.19. This branch requires polystore>=0.2.19,<0.3 and pins its submodule to the release commit. The generated configuration reference and practitioner storage explanation are updated, with a page-by-page authority review in docs/audits/pr129_tiff_compression_review_20260924.md.

Validation: 236 affected tests passed in the initial implementation pass, including real compressed TIFF readback and exact pixel equality. After the field-help repair, 15 focused documentation, MCP, and output tests passed; the documentation validator and warning-free Sphinx build passed. Following the release-floor update, the focused TIFF-output tests passed and documentation validation passed for 155 audited sources. A website-only CI import boundary was corrected by making its execution DTO import type-only; the website catalog and 23 website tests pass. Full PR CI is rerunning. No whole-plate size or runtime savings are claimed.

@trissim
trissim merged commit 2bc579c into main Sep 24, 2026
42 of 43 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.

1 participant