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

Opened 12 years ago

Closed 10 years ago

Generate modulo annotation for FLIM

Reported by: jburel Owned by: jamoore
Priority: blocker Milestone: 5.0.0-rc1
Component: Import Version: 4.4.9
Keywords: n.a. Cc: jamoore, cxallan, ajpatterson, wmoore, mlinkert, i.munro@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

Create an annotation for FLIM data when imported into OMERO
See https://www.openmicroscopy.org/site/support/file-formats/working-with-ome-xml/6d-7d-and-8d-storage

Currently we had it to channels, we may want to rethink that later.
Crucial for Imperial

Attachments (1)

Screen Shot 2013-11-18 at 13.49.14.png (115.5 KB) - added by pwalczysko 10 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 12 years ago by jburel

  • Cc wmoore mlinkert-x added; wmore removed

Discussion during satellites meeting:

  • keep storing in channels for std files and announce to the list that we will move it to t.
  • Create an XML annotation (modulo) when the image is imported.

comment:2 Changed 12 years ago by jburel

Referencing ticket #7855 has changed sprint.

comment:3 Changed 12 years ago by jburel

  • Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.1

comment:4 Changed 11 years ago by jburel

  • Cc i.munro@… added
  • Milestone changed from OMERO-4.4.x to 5.0.0-beta2
  • Version set to 4.4.9

Added support in B-F and insight to view the modulo T
The file annotation is currently only added when using the FLIMfit importer. We need to generate a annotation and link to the image in the OMERO.importer.

comment:5 Changed 10 years ago by jburel

  • Owner set to rleigh

Roger: could you have a look at that one?

comment:6 Changed 10 years ago by rleigh

Now that Melissa's work on Modulo support for CZI is merged, readers such as SDT and Lavision should be able to use the same approach as used for CZI.

comment:7 Changed 10 years ago by mlinkert

Unless I misunderstand the issue, the SDT and Lavision (Imspector) readers already generate modulo annotations as appropriate - see https://github.com/openmicroscopy/bioformats/pull/644.

comment:8 Changed 10 years ago by rleigh

If these readers are already generating correct modulo annotations, then I think I'm misunderstanding what I'm supposed to be doing for this ticket. Is the original bug resolved, or is there some additional work to do here? If so, what exactly needs doing? Thanks!

comment:9 Changed 10 years ago by jburel

if the annotation with correct namespace is generated and correctly linked to the image when imported into OMERO, for all relevant formats, we should be set.

comment:10 Changed 10 years ago by pwalczysko

Imported images from test_images_good/sdt folder using OMERO.insight-5.0.0-beta1-1586-c6ae640-ice33-b490-mac to Gretzky - crash after opening of the images happened (see https://trac.openmicroscopy.org.uk/ome/ticket/11713). The images had no Modulo Attachments after import (see screenshot).

Changed 10 years ago by pwalczysko

comment:11 Changed 10 years ago by jburel

from Ian, I just tried importing a couple of .sdt & 1 Imspector .msr files to Gretzky via insight ( 5.0.0-beta1-1586-c6ae640-ice33-b490-mac) & they seemed ti import ok & with lifetime encoded as T. However, there seemed to be no xmlAnnotation visible under "other" & ithe time-points were displayed as T which, I think means that insight couldn't find the annotation either.

comment:12 Changed 10 years ago by jburel

  • Priority changed from major to blocker

Increasing the priority since Beta2 will be rolled out at imperial.

comment:13 Changed 10 years ago by jamoore

  • Owner changed from rleigh to jamoore
  • Status changed from new to accepted

comment:14 Changed 10 years ago by jamoore

comment:15 Changed 10 years ago by jamoore

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

With the latest commit on PR 820 (https://github.com/joshmoore/bioformats/commit/b8c6415facc4871e403a6f14f39eedd31491762c) I get the following in my DB on import:

ome2=> select id, ns, textvalue from annotation order by id desc limit 1;
-[ RECORD 1 ]-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
id        | 1402
ns        | openmicroscopy.org/omero/dimension/modulo
textvalue | <Modulo namespace="http://www.openmicroscopy.org/Schemas/Additions/2011-09"><ModuloAlongT End="12304.687643793777" Start="0.0" Step="195.3125022824409" Type="lifetime" TypeDescription="TCSPC" Unit="ps"/></Modulo>

Closing for further discussion on the PR.

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

We're Hiring!