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

PrairieReader: add support for inverted X/Y coords #116

Merged
merged 1 commit into from
Aug 6, 2012

Conversation

ctrueden
Copy link
Member

@ctrueden ctrueden commented Aug 6, 2012

This works by checking the CFG file for the xYStageXPositionIncreasesLeftToRight and/or xYStageYPositionIncreasesBottomToTop properties being set to True, and then inverting all X and/or Y positions respectively if so.

With this change, Prairie data with X and/or Y coordinates inverted will still stitch properly using Fiji's "Grid/Collection stitching" plugin.

Please note, however, that there is one pending change necessary in PrairieReader before this will work completely correctly. At the moment, PrairieReader populates the OME MetadataStore before parsing the CFG file which inverts the stage coordinates. So the OME metadata is never properly adjusted with the inverted coordinates. Hence, the data will still stitch backwards by default.

Refactoring the PrairieReader to populate OME metadata after the CFG file is fully parsed will address this issue. But in the meantime, this change to PrairieReader still gets us closer to supporting the stage coordinate inversion feature.

This works by checking the CFG file for the
xYStageXPositionIncreasesLeftToRight and/or
xYStageYPositionIncreasesBottomToTop properties being set to True, and
then inverting all X and/or Y positions respectively if so.

With this change, Prairie data with X and/or Y coordinates inverted will
still stitch properly using Fiji's "Grid/Collection stitching" plugin.
melissalinkert added a commit that referenced this pull request Aug 6, 2012
PrairieReader: add support for inverted X/Y coords
@melissalinkert melissalinkert merged commit 63df591 into ome:develop Aug 6, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants