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

Opened 11 years ago

Closed 11 years ago

Bug: DeltaVision misparses exposure times

Reported by: omero-qa Owned by: mlinkert
Priority: minor Milestone: Unscheduled
Component: Bio-Formats Version: 4.4.8
Keywords: n.a. Cc: m.porter@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

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

% showinf ~/Desktop/ACA1-1000_AF594_03_R3D.dv -nopix | grep "Image.*Exposure time" | sed -e 's;.*Exposure time: ;;' | sort | uniq -c
  31 0.025000 secs
  62 0.100000 secs

% showinf ~/Desktop/ACA1-1000_AF594_03_R3D.dv -nopix -omexml -novalid | grep "ExposureTime" | sed -e 's;.*ExposureTime="\([^"]*\)".*;\1;' | sort | uniq -c
  62 1.0000000474974513E-4
  31 2.5000001187436283E-5

So the numbers here are out by a factor of 1000. The correct values would be 0.1 and 0.025 seconds, which we do have (plus floating point rounding error).

Change History (4)

comment:1 Changed 11 years ago by rleigh

The offending logic is:

        expTime = in.readFloat() / 1000;

According the the comment, expTime is in milliseconds, but the model stores the exposure time in seconds. Could the division simply be removed? Or is the unit scale different for different versions of the deltavision format?

comment:2 Changed 11 years ago by rleigh

  • Component changed from from QA to Bio-Formats

comment:4 Changed 11 years ago by mlinkert

  • Resolution set to fixed
  • Status changed from new to closed

PR merged.

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

We're Hiring!