Skip to content

Tifffile support in WSIReader#3278

Merged
Nic-Ma merged 14 commits intoProject-MONAI:devfrom
bhashemian:tifffile-support
Nov 11, 2021
Merged

Tifffile support in WSIReader#3278
Nic-Ma merged 14 commits intoProject-MONAI:devfrom
bhashemian:tifffile-support

Conversation

@bhashemian
Copy link
Copy Markdown
Member

@bhashemian bhashemian commented Nov 6, 2021

Fixes #3251

Description

This PR adds support for tifffile in WSIReader to read TIFF files (which is common in histopathology whole slide imaging). While cucim and openslide both can load patches from large whole slide images without loading the entire image into memory, tifffile loads the entire image into before extracting any patch. Thus memory consideration is needed when using tifffile for patch extraction.

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.
  • In-line docstrings updated.

@bhashemian bhashemian requested review from Nic-Ma and wyli November 6, 2021 23:24
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
@bhashemian
Copy link
Copy Markdown
Member Author

@Nic-Ma could you please take a look at this and run /build here?

@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 8, 2021

/build

Copy link
Copy Markdown
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Thanks for the update.

@Nic-Ma Nic-Ma enabled auto-merge (squash) November 11, 2021 02:26
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 11, 2021

/build

1 similar comment
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 11, 2021

/build

@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 11, 2021

/build

1 similar comment
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 11, 2021

/build

@Nic-Ma Nic-Ma merged commit cfe64aa into Project-MONAI:dev Nov 11, 2021
@bhashemian bhashemian deleted the tifffile-support branch November 23, 2021 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tifffile support in WSIReader

3 participants