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.

Sign up for GitHub

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

Jump to bottom

Handle axis offsets in czi files #876

Merged
merged 3 commits into from
Feb 21, 2023
Merged

Handle axis offsets in czi files #876

merged 3 commits into from
Feb 21, 2023

Conversation

fm3
Copy link
Member

@fm3 fm3 commented Feb 21, 2023 •
edited
Loading

Description:

  • we got some czi files where the axis start was not at 0
  • this pr adapts the czi reader to read the axis offset from the metadata, and adds this offset when reading at a position.

Todos:

  • Updated Changelog

@fm3 fm3 added the bug label Feb 21, 2023
@fm3 fm3 self-assigned this Feb 21, 2023
@fm3 fm3 requested a review from jstriebel February 21, 2023 14:28
jstriebel
jstriebel approved these changes Feb 21, 2023
View reviewed changes
Copy link
Contributor

@jstriebel jstriebel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@jstriebel jstriebel enabled auto-merge (squash) February 21, 2023 14:42
@jstriebel jstriebel merged commit 3a5aa89 into master Feb 21, 2023
@jstriebel jstriebel deleted the czi-axis-offsets branch February 21, 2023 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jstriebel jstriebel jstriebel approved these changes

Assignees

@fm3 fm3

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fm3 @jstriebel