Skip to content
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

Add lesion-wise metrics #47

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

naga-karthik
Copy link

This PR adds lesion-wise metrics to our fork of the MetricsReloaded package. This addition is to make the switch from anima to MetricsReloaded more streamlined. anima also computes lesion-wise metrics -- having the same options in MetricsReloaded would encourage a quicker adoption of the package.

valosekj and others added 11 commits February 27, 2024 16:37
Otherwise, the following KeyError is raised:

Traceback (most recent call last):
  File "/Users/user/code/MetricsReloaded/MetricsReloaded/metrics/pairwise_measures.py", line 1161, in to_dict_meas
    if len(self.measures_dict[key]) == 2:
KeyError: 'vol_diff'
…ce (RVE) in % between the prediction and the reference.
- if both reference and prediction are empty --> model learned correctly --> setting 0 representing no over- or under-segmentation
- if reference is empty, prediction is not empty --> model did not learn correctly --> setting positive value representing overestimation
Add missing `vol_diff` key to BinaryPairwiseMeasures.measures_dict
- Add info that is is a fork
- Update clone link to ivadomed
- Remove badges


Signed-off-by: Jan Valosek <[email protected]>
@naga-karthik naga-karthik changed the title Nk/add lesion wise metrics Add lesion-wise metrics May 30, 2024
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.

2 participants