Replies: 2 comments 2 replies
-
This function actually wraps any loss as part of cumulative metric https://github.com/Project-MONAI/MONAI/blob/dev/monai/metrics/loss_metric.py perhaps we can focus on this class. Meandice is not differentiable and (thus) is much faster than dice loss. |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Is your feature request related to a problem? Please describe.
Since DiceCELoss is available as a loss, it would make sense for it to be available as a metric too.
Describe the solution you'd like
Either a dedicated DiceCELoss metric like e.g. MeanDice for the DiceLoss OR a pluggable DiceCELossMetric which can be used as input to IgniteMetric.
Describe alternatives you've considered
Passing the DiceCELoss into IgniteMetric results in an error since it does not have a reset button. Which kinda makes sense, this is no metric, it does something different.
Additional context
I am also curious about why the dice got implemented at two places, namely MeanDice and the DiceLoss. From a quick glance I believe they do the same computation - so is there any reason for the code to duplicated here? The calculation logic for both should be the same.
Beta Was this translation helpful? Give feedback.
All reactions