Task #4858 (closed)
ROI related to Channels?
Reported by: | omero-qa | Owned by: | jburel |
---|---|---|---|
Priority: | minor | Milestone: | OMERO-4.4 |
Component: | Insight | Version: | n.a. |
Keywords: | n.a. | Cc: | jason@…, cxallan, saloynton, rkferguson, ajpatterson |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | 0.0d |
Sprint: | 2012-04-10 (12) |
Description
http://qa.openmicroscopy.org.uk/qa/feedback/3659/
Comment: When displaying ROIs or other objects on top of image, are these sensitive to channel? They should have different colors, depending on channel, toggle with relevant channel, etc.
Check if the ROI are linked to channels if so they should be on/off when channels change
Change History (17)
comment:1 Changed 13 years ago by jburel
- Cc cxallan added
- Component changed from from QA to Insight
- Milestone changed from Unscheduled to OMERO-Beta4.3
comment:2 Changed 13 years ago by jburel
- Sprint set to 2011-05-19 (12)
comment:3 Changed 13 years ago by jburel
- Milestone changed from OMERO-Beta4.3 to Unscheduled
- Sprint 2011-05-19 (12) deleted
comment:4 Changed 13 years ago by saloynton
Referencing ticket #3545 has changed sprint.
comment:5 Changed 12 years ago by jburel
- Cc saloynton rkferguson added
- Milestone changed from Unscheduled to OMERO-Beta4.4
- Sprint set to 2012-04-10 (12)
- Status changed from new to accepted
Show or hide shapes with channel info when the user turns on/off a channel.
comment:6 Changed 12 years ago by jburel
- Resolution set to fixed
- Status changed from accepted to closed
comment:7 Changed 12 years ago by jburel
To test:
- Since we do not set the channel for roi, you will have to edit the value directly in DB.
- Pick an image with at least 2 channels.
- set the channel to 0 for a shape.
- Turn the channel off, the shape should be hidden
- Turn the channel on, the shape should be displayed.
comment:8 Changed 12 years ago by jburel
Tested: problem if we draw a new shape the shape with channel info is still displayed.
comment:9 Changed 12 years ago by jburel
- Cc ajpatterson added
- Resolution fixed deleted
- Status changed from closed to reopened
comment:10 Changed 12 years ago by jburel
Problem is actually happening during the saving phase. The C value is reset to null.
comment:11 Changed 12 years ago by jburel
- Status changed from reopened to closed
comment:12 Changed 12 years ago by jburel
- If shape is selected and channel is turn off, the handles are still displayed.
- If channel is off and user turns the "visible" flag on. The shape is displayed.
comment:13 Changed 12 years ago by jburel
- Remaining Time set to 0.25
- Status changed from closed to reopened
various synchronization issues to fix.
comment:14 Changed 12 years ago by jburel
- Remaining Time changed from 0.25 to 0
- Resolution set to fixed
- Status changed from reopened to closed
To test:
Since we do not set the channel for roi, you will have to edit the value directly in DB.
- Pick an image with at least 2 channels.
- set the channel to 0 for a shape.
- Turn the channel off, the shape should be hidden
- Turn the channel on, the shape should be displayed.
- Go to the roi manager, after turning the channel off.
- Select the Visible box, the channel should be turned off and the shape visible.
comment:15 Changed 12 years ago by jburel <j.burel@…>
(In [386f9a53dcc60841757b63946de1017a51cb9d09/ome.git] on branch develop) Show or hide shape with channel information (see #4858)
When the channel is set for a shape, show or hide the shape
when the user turns the channel on/off.
comment:16 Changed 12 years ago by jburel <j.burel@…>
(In [4e517178fd9e66a0a58816056542d22b9fe9a3cd/ome.git] on branch develop) Do not modify the channel value for shape if set (see #4858)
comment:17 Changed 12 years ago by jburel <j.burel@…>
(In [86e029c760ab1bf2d829ffcb7785e2addcd85e67/ome.git] on branch develop) Synchronize channel selection and visibility (see #4858)
Check with Chris, those roi are not linked to a specific channel i.e. theC is not set on shape.
We should probably add support for roi with theC set, but not for 4.3