Skip to content

Changed updateSegmentationMask to handle empty segments #817

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

Merged
merged 7 commits into from
Jun 6, 2022

Conversation

marianabb
Copy link
Contributor

Fixes issue that caused labels to shift after auto segmentation when some segments are empty.

See discussion for details: #809

@SachidanandAlle SachidanandAlle added the 0.4.0 Targeted to Release version 0.4 label Jun 1, 2022
@SachidanandAlle SachidanandAlle requested a review from lassoan June 2, 2022 00:45
@SachidanandAlle
Copy link
Collaborator

@diazandr3s will help to verify and merge this PR.

Thanks for helping to fix this issue.

Copy link
Collaborator

@diazandr3s diazandr3s 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 your contribution!

@SachidanandAlle SachidanandAlle merged commit 9c46afa into Project-MONAI:main Jun 6, 2022
@marianabb marianabb deleted the handling_empty_labels branch June 7, 2022 09:38
Douwe-Spaanderman pushed a commit to Douwe-Spaanderman/MONAILabel that referenced this pull request Dec 9, 2022
…I#817)

* Changed updateSegmentationMask to handle empty segments

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: Mariana Bustamante <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: SACHIDANAND ALLE <[email protected]>
Co-authored-by: Andres Diaz-Pinto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.4.0 Targeted to Release version 0.4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants