Task #12308 (closed)
Opened 5 years ago
Closed 5 years ago
BUG: ROIs not deleting on Nightshade
| Reported by: | rkferguson | Owned by: | mporter |
|---|---|---|---|
| Priority: | critical | Milestone: | 5.1.0-m3 |
| Component: | Insight | Version: | 5.0.1 |
| Keywords: | n.a. | Cc: | ux@… |
| Resources: | n.a. | Referenced By: | https://trello.com/c/hZNSNO3h/53-bug-roi-5-0-3 |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
Michael reported that he and Sandra were working on an image with an ROI propagated across 300 time points. They deleted the ROI on 100 of the time points and this appeared to be successful, but when they ran Michael's matlab script on the image, it analysed all of the original ROIs over the 300 time points, so obviously the ROIs had not been deleted on the server.
Michael then tried a test with an image of his ID 4610 and drew and then deleted an ROI that was on it. The delete progress in the activity window continues to spin, and although the ROI has disappeared from the viewer, it was still visible when opened in another window or in the web client. I could also see it logged in as myself looking at Michael's image.
Image ID 4610 - ROI "deleted" 22/5/14 at about 1540 hrs. on Nightshade.
Change History (20)
comment:1 Changed 5 years ago by jburel
- Cc ux@… added
- Milestone changed from Unscheduled to 5.0.3
comment:2 Changed 5 years ago by mtbcarroll
- Owner changed from jburel to mtbcarroll
comment:3 Changed 5 years ago by jburel
- Referenced By set to https://trello.com/c/hZNSNO3h/53-bug-roi-5-0-3
comment:4 Changed 5 years ago by jburel
comment:5 Changed 5 years ago by jburel
Merging content of #12248. Similar problem
Bug found by Sandra using Nightshade.
Has image with 305 time points
- creates ROI and projects it across all time points
- once analysis done deletes ROI
- disappears from ROI Manager window
- creates new ROI, projects across all time points
- closes viewer
- saves changes when prompted
- new ROI is not saved
I suspect this is because the deletion of the old ROI is still being processed, so new one not actually saved before thread is killed.
Need some kind of block and warning e.g "Cannot close this window until save has been completed".
comment:6 Changed 5 years ago by jburel
Fix error while turning a line into a polyline. Problem noticed while investigating this ticket.
see https://github.com/openmicroscopy/openmicroscopy/pull/2616
comment:7 Changed 5 years ago by mtbcarroll
Conversation with Michael indicates a rarely occurring but long-standing problem in which ROI updates in Insight are not available to mtools from nightshade for many minutes.
comment:8 Changed 5 years ago by mtbcarroll
- Owner changed from mtbcarroll to jburel
I'll give this back to JM for now, as he's been looking at this code lately and so far I'm not managing to reproduce this problem at all.
comment:9 Changed 5 years ago by jburel
Discussed today with Michael, The shapes are deleted. It was explaining a delay that he noticed when creating ROI in insight (Java) and being able to access them in mtools (matlab)
Next step: write a script in java creating ROI+save and a script in matlab to check if they are immediately accessible
Balaji: did you notice some strange behaviour when creating/accessing ROI?
comment:10 Changed 5 years ago by jburel
Some ROIS were also created using mtools.
Balaji will have a look when J-M is away
comment:11 Changed 5 years ago by jburel
- Owner changed from jburel to bramalingam
comment:12 Changed 5 years ago by bramalingam
Did some independent testing (using the matlab toolbox) to see if the ROI service is getting updated in time.
There seems to be a lag (non-linear lag), potentially based on the number of operations queue'd on the server,
in retrieving the ROI's linked to the image.
Here's a small snippet of code that was used to do the test,
% Create the roi. roi = omero.model.RoiI; ellipse = createEllipse(10, 10, 5); % Indicate on which plane to attach the shape setShapeCoordinates(ellipse, 0, 0, 0); roi.addShape(ellipse); roi.setImage(omero.model.ImageI(imageId, false)); %Retrieve the roi linked to an image %%%%%%Non linear pause required at this stage , tried : pause(3)%%%%%%%%%%% service = session.getRoiService(); roiResult = service.findByImage(imageId, []); % Save iUpdate = session.getUpdateService(); roi = iUpdate.saveAndReturnObject(roi); rois = roiResult.rois; n = rois.size;
The test was able to retrieve the ROI's post this non-linear lag.
comment:13 Changed 5 years ago by jburel
Ran similar test in Java, I did not notice any issue.
comment:14 Changed 5 years ago by jburel
Ran test in Matlab too but I cannot see the problem reported by balaji.
If the measurement tool is opened, the roi created in matlab will not be visible.
comment:15 Changed 5 years ago by jburel
comment:16 Changed 5 years ago by jburel
Tested call in Java and Matlab. I have not been able to reproduce the issue.
I discussed the issue with Michael, rois are always visible in insight but not in mtools. The problem is also difficult to reproduce.
Passing it to Michael to investigate mtools
comment:17 Changed 5 years ago by jburel
- Cc jamoore added
- Owner changed from bramalingam to mporter
comment:18 Changed 5 years ago by jburel
- Cc jamoore removed
comment:19 Changed 5 years ago by jamoore
Balaji: can you explain your Matlab code above? I don't see any "saves" before your non-linear wait.
comment:20 Changed 5 years ago by jburel
- Resolution set to invalid
- Status changed from new to closed
We have not been able to reproduce the issue. Marking it as invalid. We can re-open if we see it happening again
#12248 describes the same issue.