-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
iviewer ROI Grid view #91
Comments
cc @jburel @jrswedlow @waxenegger Could this work in the absence of T-projection? |
Personally I struggle to see how this feature will work in projection view. Projection view is composed of two things: a server side projection for the image itself and showing the combined shapes for that range. The former I do not know how to update "in time". The latter concerns all not just the shapes we deal with in the grid (for the purpose of projection) but every shape in the range. So strictly speaking this is a new "mode" that would require again additional filtering on the openlayers side for shapes. |
I don't think this is a new mode, or that it involves any filtering for shapes. |
Random comment from the peanut gallery: it's nice to see (ever) more information encoded in a quickly comprehensible way. |
Following on from #88
A "Grid" view of image planes would be useful to show a heat-map of Shapes and allow multi-selection of Planes.
Could use Tabs to select regular Table of ROIs or Grid:
With Grid selected, we show a grid of planes, with the current plane highlighted:
Selecting a Range by dragging or shift-click selects multiple planes in Grid view.
We enable Z-projection to correspond to Z-range. For T-slider we simply update to the last-clicked T-index in the Grid view.
Users can also use the Z and T sliders (including Z-projection) to select plane(s) and this will sync the Grid view selection.
This will allow copying and pasting of Shapes as described in #90 without needing to wait for T-projection support from the server.
When T-projection is supported, we can also sync T-range with the range selected in the Grid view.
The text was updated successfully, but these errors were encountered: