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 #4854 (closed)

Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

BUG:Plate acquisition name

Reported by: omero-qa Owned by: ajpatterson
Priority: minor Milestone: OMERO-Beta4.3
Component: Import Version: n.a.
Keywords: n.a. Cc: jason@…, jamoore, cxallan, ajpatterson, atarkowska, wmoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-06-16 (14)

Description

http://qa.openmicroscopy.org.uk/qa/feedback/3657/

Comment: Using ScreenAcquisition?, we call these "omero.model.PlateAcquisition:0:0" This probably exposes things a bit too muhc. ;-)! Shall we call this "Plate Acquisition [N]" or something like that?


    

Change History (14)

comment:1 Changed 14 years ago by jburel

  • Milestone changed from Unscheduled to OMERO-Beta4.3

comment:2 Changed 14 years ago by jburel

  • Cc jmoore cxallan added
  • Component changed from from QA to Import
  • Owner jburel deleted

I had a closer look at the problem, The value entered at import in the DB for name is , for some plate acquisition, omero.model.plateAcquisition.

comment:3 Changed 13 years ago by bwzloranger

  • Owner set to bwzloranger

comment:4 Changed 13 years ago by bwzloranger

  • Remaining Time set to 1

comment:5 Changed 13 years ago by jburel

  • Cc ajpatterson added

Following a discussion I had with Bernhard a while ago, we came with the following sequence when displaying the "name of a plate acquisition" if not set (similar to what we have for channel Name>Fluor>Emission wavelength>index)

In the plate case Name>Start time - End time> id (not minus just separator), since they see it as "Run", we should not set term Plate acquisition for the name.
Such sequence should probably be added to the description of the Plate acquisition entity in the model, and if no name set we don't set anything for name
and let the client follow the sequence described in the model.

comment:6 Changed 13 years ago by jburel

when I say "description of the Plate Acquisition", I mean the text in the <xsd:documentation> tag.

comment:7 Changed 13 years ago by jburel

Comments?

comment:8 Changed 13 years ago by jburel

Following discussion with Chris (16/10/11), since Plate Acquisition name is no longer required in DB

  • Remove code setting default name if name not set (Brian)
  • Add logic described in a previous comment to the schema in the <xsd:documentation> (Andrew)
  • Display name (or Label) in client (insight done, Web: Ola)

comment:9 Changed 13 years ago by jburel

  • Cc atarkowska wmoore added

comment:10 Changed 13 years ago by bwzloranger <bwzloranger@…>

(In [a270c3596c8f305a4b3ba233eab92221fce893e6/ome.git] on branch develop) See #4854. Plate acquisition code was removed (as pre request) from the importer. Should now be handled by xsd and display label in insite/web

comment:11 Changed 13 years ago by bwzloranger

  • Owner changed from bwzloranger to ajpatterson

comment:12 Changed 13 years ago by ajpatterson

  • Remaining Time changed from 1 to 0.1
  • Sprint set to 2011-06-16 (14)
  • Status changed from new to accepted

comment:13 Changed 13 years ago by ajpatterson

  • Remaining Time changed from 0.1 to 0
  • Status changed from accepted to closed
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.68758 sec.)

We're Hiring!