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)
Change History (16)
comment:1 Changed 12 years ago by jburel
- Cc wmoore mlinkert-x added; wmore removed
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
PR opened, but still needs work: https://github.com/openmicroscopy/bioformats/pull/820
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.
Discussion during satellites meeting: