Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for deprecated TiffWriter.save method. #12

Merged
merged 4 commits into from May 31, 2022

Conversation

agvaughan
Copy link
Contributor

TiffWriter.save is deprecated - we should replace replace with TiffWriter.write.
Issue: #11

I also added a separate test.yml workflow to make development easier when not intending to deploy.

Changelog from tifffile

2022.2.2
Fix TypeError when second ImageDescription tag contains non-ASCII (#112). Fix parsing IJMetadata with many IJMetadataByteCounts (#111). Detect MicroManager NDTiffv2 header (not tested). Remove cache from ZarrFileSequenceStore (use zarr.LRUStoreCache). Raise limit on maximum number of pages. Use J2K format when encoding JPEG2000 segments. Formally deprecate imsave and TiffWriter.save. Drop support for Python 3.7 and numpy < 1.19 (NEP29).

@agvaughan
Copy link
Contributor Author

Passes tests (and works on my machine).

@jwindhager jwindhager self-requested a review May 31, 2022 11:11
@jwindhager jwindhager self-assigned this May 31, 2022
@jwindhager
Copy link
Contributor

Thanks for this! Will include it in today's maintenance release

@jwindhager jwindhager merged commit d7daeae into BodenmillerGroup:master May 31, 2022
@agvaughan
Copy link
Contributor Author

Thanks! Appreciate it.

1 similar comment
@agvaughan
Copy link
Contributor Author

Thanks! Appreciate it.

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.

None yet

2 participants