Skip to content

Pathology Probability Map Generator#1893

Merged
bhashemian merged 15 commits intoProject-MONAI:masterfrom
bhashemian:prob_map_generator
Mar 31, 2021
Merged

Pathology Probability Map Generator#1893
bhashemian merged 15 commits intoProject-MONAI:masterfrom
bhashemian:prob_map_generator

Conversation

@bhashemian
Copy link
Copy Markdown
Member

Description

This PR implements the handler requires to get pathology patch inference outputs, generate probability maps, and save them.

Status

Work in progress

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • 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.
  • Documentation updated, tested make html command in the docs/ folder.

@bhashemian bhashemian changed the title [WIP] Pathology Probability Map Generator Pathology Probability Map Generator Mar 29, 2021
Copy link
Copy Markdown
Contributor

@wyli wyli left a comment

Choose a reason for hiding this comment

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

thanks!

Copy link
Copy Markdown
Contributor

@wyli wyli left a comment

Choose a reason for hiding this comment

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

thanks, looks good to me!

@bhashemian bhashemian enabled auto-merge (squash) March 31, 2021 14:14
Copy link
Copy Markdown
Contributor

@yiheng-wang-nv yiheng-wang-nv left a comment

Choose a reason for hiding this comment

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

Thanks, it looks fine.

@bhashemian bhashemian merged commit 972fa5f into Project-MONAI:master Mar 31, 2021
@bhashemian bhashemian deleted the prob_map_generator branch April 12, 2021 20:22
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