ome / design

OME Design proposals
http://ome.github.io/design/
1 stars 15 forks source link

iviewer Shape propagation #88

Open will-moore opened 6 years ago

will-moore commented 6 years ago

Major feature supported by Insight but not by iviewer 1.0.0.

See previous ideas at https://github.com/openmicroscopy/design/issues/59

Goals:

Suggestions:

Here, we add a "ROI Grid" option under Edit (could also put this option under a right-click menu on ROI/Shape).

roi-grid1

This displays the shapes under ROI as in the Insight grid.

roi-grid2

roi-grid3

Other thoughts:

will-moore commented 6 years ago

cc @waxenegger @pwalczysko @jburel @mporter-gre Feedback & thoughts welcome!

pwalczysko commented 6 years ago

If we ensure that a contiguous range is selected ...

Does that mean that the user can paste the ROI only on contiguous range of z/t ? I hope not.

pwalczysko commented 6 years ago

I would add an axis legend to the grid (z, t). It is hard to realize what the grid actually shows otherwise.

pwalczysko commented 6 years ago

ROI grid - if the user does not know upfront what that is, there is no idea about why and how this could be useful. Wouldn't it be better to use the word propagate ?

will-moore commented 6 years ago

Does that mean that the user can paste the ROI only on contiguous range of z/t ? I hope not.

Yes - this is the current behaviour in Insight - I can only propagate to a contiguous set of planes at a time. The ROI grid approach is actually more flexible since I can Copy and Paste to any planes, even if they are not contiguous with the original plane.

Adding an axis makes sense, agreed.

I chose "ROI grid" because it is really just a different layout and is not only for propagation. You could simply use it as a different way of seeing the distribution of Shapes in an ROI. Actually, it might be useful to enable the same view of ALL shapes on the image, not just a single ROI. This would need more thought about how to handle display, copy and paste etc, but the view itself could be handy.

The propagation happens simply because you are able to select more than 1 plane to paste the shapes to (and we also know to put all the shapes in the same ROI). However, it could also be nice to allow propagation without ROI grid by copying a shape, selecting a range of planes (using Z-projection) and pasting to all those planes. The only question in this case would be "Do we put all these shapes in the same ROI or separate ROIs?".

will-moore commented 6 years ago

@pwalczysko It may be that "ROI grid" is not the best term (especially since it will be a single "row" or "column" for most images). I just don't want to pick the wrong name simply to align with Insight behaviour, but it might be that "propagate" is actually the best in this case.

jburel commented 6 years ago

The term "grid" leads to something different for me, a feature requested by user, the ability to "browse" an image by plane, we could misuse the multiview for that and put each plane in a grid. That's for a different scoping issue We should avoid the term grid for propagation

will-moore commented 6 years ago

After more discussion with @pwalczysko and @jburel, I think this is diverging into two independent features:

I'll describe these latest ideas on a new issue, to avoid confusion with those above.