bug, IJ1: contours and ROI manager

[xulman]

reported originally outside:

1. Problem when closing the RoiManager
2. When uncheck “Add to Roi” SAMJ doesn’t generate a outline of the segmentation

The issue with the latter is that contours are driven through the ROI manager, so if one disables adding new contours into the manager, the new ones don't get displayed... we have to explicitly draw them via IJ1 API (non ROI manager API).

[carlosuc3m]

Hello @xulman I already implemented a fast temporal fix (roi manager cannot be unchecked lol).

I think this issue is a little bit deeper and that we need to figure out what behaviour we actually want from the ROI manager, because now it is really not defined.

[xulman]

I think this issue is a little bit deeper and that we need to figure out what behaviour we actually want from the ROI manager, because now it is really not defined.

yes, we especially need to think outside ROI manager and IJ1 prompts, not always one would want to interact with SAMJ via these (consider BigDataViewer as an gui interface, the macro scripts, etc..). For BDV, for example, we would probably need to come up with an own dialog that reviews the current segmentation countours -- maybe we would use it for IJ1 too... as the functionality/possibilities of ROI manager is not exactly matching SAMJ (which is another source of confusion for users)