DeepRegNet / DeepReg

Medical image registration using deep learning
Apache License 2.0
564 stars 76 forks source link

Fix Dice Score #734

Closed mathpluscode closed 3 years ago

mathpluscode commented 3 years ago

Subject of the issue

@YipengHu obeserved dice score > 1, which is clearly a bug.

It might be caused by https://github.com/DeepRegNet/DeepReg/commit/3a457eef313f4bee61d16e899d15a87e6c831e91. where we somehow changed reduce_sum to mean, but as we are adding EPS to the numerator/denominator, the reduce mean may lead to values << 1 and therefore the result is no longer accurate.

YipengHu commented 3 years ago

changing to sum still has dice score greater than 1 (see below the BinaryDiceScore on val) 2021-04-02 (2)