Conversation
|
Update: I discussed with @KumoLiu , and according to my understand of the original paper, the predictions should only have 1 channel, since for each pixel, it should only belong to one instance. Therefore, the ground truth of an image with shape |
|
/black |
Hi @yiheng-wang-nv, thanks for implementing PQ metrics. You are right, in the paper, the PQ metrics is calculated per class for the input is one dimensional for each PQ calculation; however, these PQs are averaged afterwards to create the metrics that we can denote as Unlike the rest of MONAI metrics that are designed for semantic segmentation, this metrics is used for instance segmentation and they can have multiple channels. Having said that, it still depends how the data is being represented, for instance look at the ground truth of this dataset, where one channel is for instances and one channel for classes: https://conic-challenge.grand-challenge.org/Data/ I believe we can have two classes, one for binary PQ metrics where it assumes only one-channel inputs, and another for multi-class PQ where it accepts two channels (one for instance and one for classes) and you call the binary PQ inside it for each class. Please let me know if you have any question. |
|
Hi @yiheng-wang-nv, please let me know when this PR is ready for review. Thanks |
|
@yiheng-wang-nv, it would be great if this can output SQ (segmentation quality) and RQ (recognition quality) in addition to PQ which is the combination of them. Thanks |
591f1f5 to
3f82016
Compare
Signed-off-by: Yiheng Wang <[email protected]>
|
Hi @drbeh , I modified the metric implementation to align with CoNiC challenge's version, and if calling Could you please help to re-review it? Thanks, and if the format meets the requirements, I'll add unit tests asap. |
Co-authored-by: Dženan Zukić <[email protected]>
Signed-off-by: Yiheng Wang <[email protected]>
Signed-off-by: Yiheng Wang <[email protected]>
80494a7 to
29e476e
Compare
DCO Remediation Commit for Yiheng Wang <[email protected]> I, Yiheng Wang <[email protected]>, hereby add my Signed-off-by to this commit: 6984b83 Signed-off-by: Yiheng Wang <[email protected]> Co-authored-by: Wenqi Li <[email protected]>
Signed-off-by: Yiheng Wang <[email protected]>
Signed-off-by: Yiheng Wang <[email protected]>
|
/black |
|
/build |
Signed-off-by: Yiheng Wang <[email protected]>
|
/build |
bhashemian
left a comment
There was a problem hiding this comment.
Based on our today's conversation, I am reviewing it. Please let me finish it first.
bhashemian
left a comment
There was a problem hiding this comment.
Hi @yiheng-wang-nv, nice work! I left few comments and it would be great if you can address them. Thanks
Signed-off-by: Yiheng Wang <[email protected]>
Signed-off-by: Yiheng Wang <[email protected]>
|
/black |
|
/build |
Signed-off-by: monai-bot <[email protected]>
|
/build |
|
/build |
Fixes Project-MONAI#5344 . ### Description This PR implements the metric of Panoptic Quality. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [x] 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 --disttests`. - [x] In-line docstrings updated. - [x] Documentation updated, tested `make html` command in the `docs/` folder. Signed-off-by: Yiheng Wang <[email protected]> Co-authored-by: Behrooz Hashemian <[email protected]> Co-authored-by: Dženan Zukić <[email protected]> Co-authored-by: Wenqi Li <[email protected]> Signed-off-by: Behrooz <[email protected]>
Fixes #5344 .
Description
This PR implements the metric of Panoptic Quality.
Types of changes
./runtests.sh -f -u --net --coverage../runtests.sh --quick --unittests --disttests.make htmlcommand in thedocs/folder.