Task #7856 (closed)
Opened 8 years ago
Closed 6 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 7 years ago by jburel
- Cc wmoore mlinkert-x added; wmore removed
comment:2 Changed 7 years ago by jburel
Referencing ticket #7855 has changed sprint.
comment:3 Changed 7 years ago by jburel
- Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.1
comment:4 Changed 6 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 6 years ago by jburel
- Owner set to rleigh
Roger: could you have a look at that one?
comment:6 Changed 6 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 6 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 6 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 6 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 6 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 6 years ago by pwalczysko
comment:11 Changed 6 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 6 years ago by jburel
- Priority changed from major to blocker
Increasing the priority since Beta2 will be rolled out at imperial.
comment:13 Changed 6 years ago by jamoore
- Owner changed from rleigh to jamoore
- Status changed from new to accepted
comment:14 Changed 6 years ago by jamoore
PR opened, but still needs work: https://github.com/openmicroscopy/bioformats/pull/820
comment:15 Changed 6 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: