Current implementation of POGM in pysap-mri does not pass the linear_op to the optimizer due to which it is internally set to Identity.
However, as POGM carries out complete optimization on the wavelet coefficients, the SSIM metric calculations needs the adjoint operation which is performed here:
However, for this, the linear operator must be set right which isnt done right in current scenario.
This prevents faster iterations of retrospective studies and also a better grid search.
Tagging as a bug to just raise the importance of the issue.
Current implementation of POGM in pysap-mri does not pass the linear_op to the optimizer due to which it is internally set to
Identity. However, as POGM carries out complete optimization on the wavelet coefficients, the SSIM metric calculations needs the adjoint operation which is performed here:https://github.com/CEA-COSMIC/ModOpt/blob/f9aa7b6154bfe34bc79d94d3c7d03350d47eef4a/modopt/opt/algorithms.py#L1347-L1356
However, for this, the linear operator must be set right which isnt done right in current scenario. This prevents faster iterations of retrospective studies and also a better grid search.
Tagging as a bug to just raise the importance of the issue.