Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Leica LIF: fix multi-dimensional data from color cameras #2313

Merged
merged 1 commit into from Apr 13, 2016

Conversation

melissalinkert
Copy link
Member

Fixes https://trac.openmicroscopy.org/ome/ticket/13112.

This fixes the case when a color camera is used (so each plane has 3 RGB
channels), and multiple channels are acquired potentially in addition to
multiple timepoints and/or Z sections.

To test, use the files from QA 16911 and QA 16919. Compare the images as shown in Leica's LAS X or LAS AF Lite software against what is shown by ImageJ with and without this change. Also verify that with this change, no error messages are shown when opening either file in ImageJ.

Fixes #13112.

This fixes the case when a color camera is used (so each plane has 3 RGB
channels), and multiple channels are acquired potentially in addition to
multiple timepoints and/or Z sections.
@melissalinkert
Copy link
Member Author

@dgault
Copy link
Member

dgault commented Apr 13, 2016

Code changes make sense and the two QA files now open without any of the errors that were thrown before applying the PR. The images look and display as expected also. Config PR has also been tested and is green - https://ci.openmicroscopy.org/view/Bio-Formats-DEV/job/BIOFORMATS-DEV-merge-repository-subset/207/

Changes look good to merge.

@sbesson sbesson merged commit c19042c into ome:develop Apr 13, 2016
@sbesson sbesson added this to the 5.2.0 milestone Apr 19, 2016
@melissalinkert melissalinkert deleted the 13112 branch January 9, 2017 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants