jleuschn / dival

Deep Inversion Validation Library
MIT License
74 stars 13 forks source link

Iterative reconstructors wrapping ODL solvers do not use hyper parameter `'iterations'` #38

Open jleuschn opened 3 years ago

jleuschn commented 3 years ago

The iterative reconstructor in from dival.reconstructors.odl_reconstructors inherit from dival.reconstructors.reconstructor.IterativeReconstructor, and thus should reflect the hyper parameter iterations. Instead they currently use an unrelated attribute niter. This should be merged, ideally ensuring backward compatibility.