Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

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

comment:4 Changed 5 years ago by jburel

#12248 describes the same issue.

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.

Last edited 5 years ago by bramalingam (previous) (diff)

comment:13 Changed 5 years ago by jburel

Ran similar test in Java, I did not notice any issue.

Last edited 5 years ago by jburel (previous) (diff)

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

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.96462 sec.)

We're Hiring!