Skip to content

Conversation

@LukasMicroscopy
Copy link

This includes the 8bit nd tiff tag bugfix, put_image lock for thread safety, reader optimization (mmap) and zip (zlib) compression.
The compression tag value was chosen to be the same like in tiff specifications (1 for no compression, 8 for zip/zlib compression).
The compression is compatible with standard tiff readers.
For now there are no zlib options (uses zlib standard keys).

ELEVEN_BIT_MONOCHROME = 6

ZLIB_COMPRESSED = 8
NO_COMPRESSION = 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

This seems redundant to UNCOMPRESSED. If NO_COMPRESSION is a standard name, maybe good to remove uncompressed

@henrypinkard
Copy link
Collaborator

I'd suggest merging this (assuming it passes test). However, my write access to this repository has been removed. Perhaps @nicost or @marktsuchida could merge it?

@marktsuchida
Copy link
Member

Thanks, @henrypinkard! That would be the effect of you retiring from the org, but we can add you as a collaborator on specific repos. I've sent an invite.

@henrypinkard
Copy link
Collaborator

Thanks @marktsuchida . I still seem to not have permissions to approve running the tests

@marktsuchida
Copy link
Member

@henrypinkard Strangely, I don't see a button to approve either. (Never seen "Waiting for status to be reported" before.)
I gave you admin for this repo. Does that make any difference?

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.

3 participants