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 #11293 (accepted)

Opened 11 years ago

Last modified 10 years ago

Disallow unparsable timestamp — at Version 5

Reported by: omero-qa Owned by: ajpatterson
Priority: major Milestone: OMERO-5
Component: Specification Version: 5.0.0-beta1
Keywords: n.a. Cc: m.porter@…, jamoore, rleigh
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description (last modified by ajpatterson)

Create a new OME timestamp type that is a tighter restriction on the standard XSD timestamp. This will limit the object to what the code libraries we have available can actually process. Mostly this is limiting seconds to three decimal places.

The acceptable formats will follow these patterns

1111-11-11T11:11:11.111
1111-11-11T11:11:11.111Z
1111-11-11T11:11:11.111+11:11
1111-11-11T11:11:11.111-11:11
i.e.
YYYY-MM-DDThh:mm:ss.sss+HH:zz
Year
Month
Day
hour
minute 
second (with up to three decimal places)
Hour of time zone
z - minute of time zone

Was: Bug: Model supports only second resolution

https://www.openmicroscopy.org/qa2/qa2/qa/feedback/7443/

Comment: The little coloured squares at the bottom of an image reporting the acquisition time for each channel don't work for images less than one second. So in a 3 channel image I only get 2 values (for green and red) for the first 6 z-sections with the last value (blue) appearing after that. I guess this is because of the values being rounded down to the nearest second, but it does look a bit odd.

AcquisitionTime? and TimeIncrement? have only second resolution; it's not possible for higher resolution timing to be stored at present.

Change History (5)

comment:1 Changed 11 years ago by rleigh

  • Component changed from from QA to Model

comment:2 Changed 11 years ago by ajpatterson

Schema does not need changed.

xsd:dateTime already has arbitrary precision for seconds.

see: http://www.w3.org/TR/xmlschema-2/#isoformats

Functions in the OME Model API will need updated.

Display code in OMERO.insight will need updated.

comment:3 Changed 11 years ago by ajpatterson

  • Owner changed from ajpatterson to rleigh

comment:4 Changed 11 years ago by jamoore

  • Component changed from Model to Specification
  • Milestone changed from Unscheduled to OMERO-5
  • Owner changed from rleigh to ajpatterson
  • Priority changed from minor to major
  • Version changed from 4.4.8 to 5.0.0-beta1

This isn't a problem with the specification per-se, but there are no libraries for supporting all of the variants:

  • In 4.4, we could put in a documentation change, not necessarily.
  • In 5.x, we should disallow timestamps that aren't parse-able in our coding languages.

comment:5 Changed 11 years ago by ajpatterson

  • Cc jamoore rleigh added
  • Description modified (diff)
  • Status changed from new to accepted
  • Summary changed from Bug: Model supports only second resolution to Disallow unparsable timestamp
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.70301 sec.)

We're Hiring!