Skip to content

Clarification required for SurfaceDiceMetric #5906

@tvercaut

Description

@tvercaut

Describe the bug
The documentation of the SurfaceDiceMetric is confusing and would benefit from being clarified:
https://github.com/Project-MONAI/MONAI/blob/dev/monai/metrics/surface_dice.py

It indeed says it computes the "Normalized Surface Distance (NSD)" intially and later it says it computes the "(Normalized) Surface Dice (NSD)". It then refers to https://github.com/deepmind/surface-distance which does not provide a metric called "Normalized Surface Distance" or "Normalized Surface Dice".

The DeepMind repo however provides a Surface Dice (with compute_surface_dice_at_tolerance) which may be what SurfaceDiceMetric implements but compute_surface_dice_at_tolerance requires a tolerance as input whereas SurfaceDiceMetric does not. It's thus unclear how these can be the same.

The referenced paper in MONAI's documentation, https://arxiv.org/abs/1809.04430, also does not mention a metric which accronym would be NSD. The reference for the 2D extension does though: https://arxiv.org/abs/2111.05408

Additional context
Similar confusions can be found in biomedical challenges:
https://doi.org/10.1038/s41467-022-30695-9
https://doi.org/10.1016/j.media.2020.101920

It's thus not only MONAI being confusing but it would be helpful to clarify nonetheless.

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions