ROI auto detection method should be moved from UI
I tolerated the AdjustROI
inside the Adjust
class earlier because Adjust
itself was a "relatively" short-ish class. Now it started to feel crowded and instead of asking you to extract some of the newly added logging I figured it would be easier to mention that old thing. Ideally, I see the detect_analysis_region
method as a candidate to be extracted under either the application or domain package - I would like to see less math in UI classes.