You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Originally posted by thibaultdvx September 13, 2024
Hi,
I would like to suggest to add the Average Precision in the classification metrics. I think that the implementation would be very similar to ROC AUC, so it wouldn't be a big deal to integrate it.
I would be happy to make a PR to do so.
I also feel that an important regression metric is missing:R2 score. Is there any reason why?
Once again, I would be happy to help to implement it!
WDYT?
The text was updated successfully, but these errors were encountered:
FixesProject-MONAI#8085.
### Description
Average Precision is very similar to ROCAUC, so I was very much inspired
by the ROCAUC implementation.
More precisely, I created:
- `AveragePrecisionMetric` and `compute_average_precision` in
`monai.metrics`,
- a handler called `AveragePrecision` in `monai.handlers`,
- three unittest modules: `test_compute_average_precision.py`,
`test_handler_average_precision.py` and
`test_handler_average_precision_dist.py`.
I also modified the docs to mention Average Precision.
### 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).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [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: thibaultdvx <thibault.devarax@icm-institute.org>
Signed-off-by: Thibault de Varax <154365476+thibaultdvx@users.noreply.github.com>
Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
Co-authored-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>
Signed-off-by: Can-Zhao <volcanofly@gmail.com>
Discussed in #8081
Originally posted by thibaultdvx September 13, 2024
Hi,
I would like to suggest to add the Average Precision in the classification metrics. I think that the implementation would be very similar to ROC AUC, so it wouldn't be a big deal to integrate it.
I would be happy to make a PR to do so.
I also feel that an important regression metric is missing: R2 score. Is there any reason why?
Once again, I would be happy to help to implement it!
WDYT?
The text was updated successfully, but these errors were encountered: