Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
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 #9881 (closed)

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

Bug: Acquisition Rating etc

Reported by: wmoore Owned by: wmoore
Priority: major Milestone: OMERO-4.4.7
Component: Insight Version: n.a.
Keywords: n.a. Cc: kgillen, cblacburn, jburel
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2012-12-18 (3)

Description (last modified by wmoore)

Kenny and I have seen some strange behaviour when editing ratings on Plate-Acquisitions.

Seems like there's some very inconsistent updating of the right-hand panel.

E.g Rate several Acquisitions differently, then select one after the other. Each time I select a different one I get a different rating - not consistently.

Then I got a strange error:
NB: using webstart Insight:

java.lang.Exception: Abnormal termination due to an uncaught exception.
java.lang.NoSuchMethodError: net.n3.nanoxml.IXMLElement.getChildrenNamed(Ljava/lang/String;)Ljava/util/ArrayList;
	at org.openmicroscopy.shoola.agents.util.editorpreview.PreviewModel.parseXmlDescription(PreviewModel.java:95)
	at org.openmicroscopy.shoola.agents.util.editorpreview.PreviewModel.<init>(PreviewModel.java:143)
	at org.openmicroscopy.shoola.agents.util.editorpreview.PreviewPanel.setDescriptionXml(PreviewPanel.java:358)
	at org.openmicroscopy.shoola.agents.util.editorpreview.PreviewPanel.<init>(PreviewPanel.java:341)
	at org.openmicroscopy.shoola.agents.metadata.editor.DocComponent.formatTootTip(DocComponent.java:389)
	at org.openmicroscopy.shoola.agents.metadata.editor.DocComponent.initComponents(DocComponent.java:591)
	at org.openmicroscopy.shoola.agents.metadata.editor.DocComponent.<init>(DocComponent.java:743)
	at org.openmicroscopy.shoola.agents.metadata.editor.DocComponent.<init>(DocComponent.java:755)
...

Attachments (3)

Screen shot 2012-11-08 at 12.57.54.png (60.3 KB) - added by wmoore 11 years ago.
Web shows all ratings on PlateAcquisition?
9881-fix.png (170.7 KB) - added by jburel 11 years ago.
9881-web-afterfix.png (248.4 KB) - added by jburel 11 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 11 years ago by wmoore

  • Description modified (diff)

comment:2 Changed 11 years ago by wmoore

Seems each Plate-Acquisition is linked to multiple ratings:

omero=# select * from PlateAcquisitionAnnotationLink order by id desc;
 id  | permissions | version | child | creation_id | external_id | group_id | owner_id | update_id | parent 
-----+-------------+---------+-------+-------------+-------------+----------+----------+-----------+--------
 116 |        -120 |         |  5018 |      142345 |             |        3 |        3 |    142345 |      6
 115 |        -120 |         |  5017 |      142343 |             |        3 |        3 |    142343 |      5
 114 |        -120 |         |  5016 |      142341 |             |        3 |        3 |    142341 |      5
 113 |        -120 |         |  5015 |      142339 |             |        3 |        3 |    142339 |      5
 112 |        -120 |         |  5014 |      142337 |             |        3 |        3 |    142337 |      5
 111 |        -120 |         |  5013 |      142335 |             |        3 |        3 |    142335 |      4
 110 |        -120 |         |  5012 |      142333 |             |        3 |        3 |    142333 |      4
 109 |        -120 |         |  5011 |      142331 |             |        3 |        3 |    142331 |      6
 108 |        -120 |         |  5010 |      142329 |             |        3 |        3 |    142329 |      5
 107 |        -120 |         |  5009 |      142327 |             |        3 |        3 |    142327 |      5
 106 |        -120 |         |  5008 |      142325 |             |        3 |        3 |    142325 |      4
 105 |        -120 |         |  5007 |      142323 |             |        3 |        3 |    142323 |      4
 104 |        -120 |         |  5006 |      142321 |             |        3 |        3 |    142321 |      4
 103 |        -120 |         |  5005 |      142319 |             |        3 |        3 |    142319 |      4
 102 |        -120 |         |  5004 |      142317 |             |        3 |        3 |    142317 |      4
 101 |        -120 |         |  5003 |      142315 |             |        3 |        3 |    142315 |      4

omero=# select id, longvalue from annotation order by id desc;
  id  | longvalue 
------+-----------
 5018 |         5
 5017 |         3
 5016 |         3
 5015 |         3
 5014 |         3
 5013 |         5
 5012 |         5
 5011 |         2
 5010 |         1
 5009 |         4
 5008 |         3
 5007 |         3
 5006 |         3
 5005 |         3
 5004 |         2
 5003 |         1

Changed 11 years ago by wmoore

Web shows all ratings on PlateAcquisition?

comment:3 Changed 11 years ago by jburel

it does not really make sense to show all the ratings, but as discussed that will probably be removed/replaced

comment:4 Changed 11 years ago by jburel

For the exception, we need to decide what to do with editor or at list with the library used. That's not the first we see such error b/c of nanoxml

comment:5 Changed 11 years ago by jburel

  • Sprint changed from 2012-11-20 (2) to 2012-12-18 (3)

Moved from sprint 2012-11-20 (2)

comment:6 Changed 11 years ago by jburel

see #9926

Using web start Insight on howe I get a crash if I try to select the following image:

user: user-3 
group: read-annotate-1 
dataset: avi (id=61)
image: Level003-DB-089.avi

This was initially put down to bad data but checking the same image with the webclient I can select the image and view it with no problems.

Exception attached in comment.

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

comment:7 Changed 11 years ago by jburel

  • Cc cblacburn bpindelski added

The problem is due to the fact that the jars ordering described in the ivy file is not the order used when using insight/webstart.
Basically it is important to have jhotdraw in the class path before nanoxml (library used by the editor) since jhotdraw uses a different version of nanoxml.

Will: how long will it take to get rid of nanoxml since the editor is no longer under development.
We also need to check what we can do for the webstart build. (might be quicker)

comment:8 Changed 11 years ago by jburel

  • Remaining Time set to 0.15
  • Status changed from new to accepted

comment:9 Changed 11 years ago by jmoore

See grep output in #9926 for one possible solution

Changed 11 years ago by jburel

Changed 11 years ago by jburel

comment:10 Changed 11 years ago by jburel

  • Owner changed from jburel to wmoore

Problem has now been fixed in insight but the rating no longer shows up in web.
Testing against howe/user1
Screenshots attached to the ticket.
Code https://github.com/jburel/openmicroscopy/tree/rating-9881
Will I am passing the ticket to you. If you need to make some changes it probably makes more sense to use my branch. You can open PR against it when ready.

comment:11 Changed 11 years ago by jburel

  • Cc jburel added; bpindelski removed

comment:12 Changed 11 years ago by wmoore

  • Remaining Time changed from 0.15 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

comment:13 Changed 11 years ago by jburel <j.burel@…>

(In [1abe219b0e3c8e8b833ceeff0d6ff5b8f8d330ba/ome.git] on branch develop) Save one rating per object for plate acquisition (see #9881)

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.83397 sec.)

We're Hiring!