-
Notifications
You must be signed in to change notification settings - Fork 739
Update Multiple Instance Learning Pipeline #728
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
Conversation
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
for more information, see https://pre-commit.ci
@Nic-Ma Here is the accuracy metric of the updated pipeline with respect to the original one that Andriy has run the experiment and produced the figures. The dark blue is the original one and the orange is the updated one (ignore the other colors). Both dark blue and orange ones are performing very similarly considering the different randomness that they have. |
Signed-off-by: Behrooz <[email protected]>
Signed-off-by: Behrooz <[email protected]>
pathology/multiple_instance_learning/panda_mil_train_evaluate_pytorch_gpu.py
Outdated
Show resolved
Hide resolved
pathology/multiple_instance_learning/panda_mil_train_evaluate_pytorch_gpu.py
Show resolved
Hide resolved
Signed-off-by: Behrooz <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update, looks good to me.
@wyli Do you have any other comments?
Thanks.
* Update mil pipeline Signed-off-by: Behrooz <[email protected]> * Fix a typo Signed-off-by: Behrooz <[email protected]> * Tiffile to cucim Signed-off-by: Behrooz <[email protected]> * Update to RandGridPatchd Signed-off-by: Behrooz <[email protected]> * Fix import Signed-off-by: Behrooz <[email protected]> * Add sort_key Signed-off-by: Behrooz <[email protected]> * sort key max to min Signed-off-by: Behrooz <[email protected]> * Fix patch_size Signed-off-by: Behrooz <[email protected]> * Stack patch locations Signed-off-by: Behrooz <[email protected]> * Remove location Signed-off-by: Behrooz <[email protected]> * Change to fix_num_patches Signed-off-by: Behrooz <[email protected]> * Update grid patch Signed-off-by: Behrooz <[email protected]> * Add threshold_filter Signed-off-by: Behrooz <[email protected]> * Update threshold and add pad_mode=None Signed-off-by: Behrooz <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update mil pipeline with new grid patch Signed-off-by: Behrooz <[email protected]> * Update GridPatch args Signed-off-by: Behrooz <[email protected]> * Update LabelEncodeIntegerGraded and sort imports Signed-off-by: Behrooz <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Nic Ma <[email protected]>
Fixes #637
Description
This PR updates Multiple Instance Learning Pipeline with the new components in MONAI.
Status
Ready
Checks
./runner [-p <regex_pattern>]