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

Add tile padding as an option #70

Merged
merged 2 commits into from
Sep 9, 2024
Merged

Conversation

petebankhead
Copy link
Member

@petebankhead petebankhead commented Sep 9, 2024

This adds 'Tile padding' as an additional option.
This relates to #66

Ideally, we'd like to compute this automatically and avoid requesting it from the user. But we're still improving the tiling code, and at this point it's useful to be able to change the padding.

As a rule:

  • Lower padding values are faster (especially when the tile size is also low)
  • Higher padding values are necessary if you see objects being split/fragmented/otherwise weird on tile boundaries

In an image where all the nuclei are small (e.g. CMU-1.svs), a padding of 16 might be fine. But on an image with some huge nuclei (e.g. OS-2.ndpi), a larger value (e.g. 80) might be needed to avoid missing important cells.

@petebankhead petebankhead merged commit 0fc6355 into qupath:main Sep 9, 2024
@petebankhead petebankhead deleted the tile-padding branch September 9, 2024 12:01
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