Closed tkoskela closed 1 week ago
MPI communicators adjoint_space_comm and direct_space_comm in imaging_forward_backward.h were removed during the refactor in #303. These should go back in, because they are needed by https://github.com/astro-informatics/sopt/blob/ae4c48e397b99d6b27b727744da292cb3f1817ce/cpp/sopt/l1_proximal.h#L151-L156.
adjoint_space_comm
direct_space_comm
If the l1_proximal class is the only class that uses them, we can make them members of the l1_g_proximal class and set them during initialisation in purify, we don't need a pulblic interface in the parent g_proximal class.
l1_proximal
l1_g_proximal
purify
g_proximal
Closed by https://github.com/astro-informatics/purify/pull/315
MPI communicators
adjoint_space_comm
anddirect_space_comm
in imaging_forward_backward.h were removed during the refactor in #303. These should go back in, because they are needed by https://github.com/astro-informatics/sopt/blob/ae4c48e397b99d6b27b727744da292cb3f1817ce/cpp/sopt/l1_proximal.h#L151-L156.If the
l1_proximal
class is the only class that uses them, we can make them members of thel1_g_proximal
class and set them during initialisation inpurify
, we don't need a pulblic interface in the parentg_proximal
class.