facebookresearch / fastMRI

A large-scale dataset of both raw MRI measurements and clinical MRI images.
https://fastmri.org
MIT License
1.3k stars 372 forks source link

validation ssim fix #124

Closed z-fabian closed 3 years ago

z-fabian commented 3 years ago

evaluate.ssim receives a 2D array when a 3D array is expected with a slice dimension first. This results in wrong validation SSIM, which is fixed by adding a singleton dimension to the recons and targets.

Maybe a dimension check in evaluate.ssim would help avoiding this bug later?

z-fabian commented 3 years ago

Updated!