-
Notifications
You must be signed in to change notification settings - Fork 17
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
When loading data for train allow to read data that has channel on first position #94
Conversation
Thanks for starting to work on this @Czaki. I should have time to have a look at this later in the evening or tomorrow. |
In this PR, currently, there is no metadata parsing. Only guessing based on the size of the array. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this! This is good to go once the tests pass (currently just missing a tempfile
import.)
@@ -102,17 +102,23 @@ def _get_sample(self, index): | |||
|
|||
shape = raw.shape | |||
# we assume images are loaded with channel last! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's update this comment since we now decide on channel last vs. channel first based on the image shape.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you check the current version?
Co-authored-by: Constantin Pape <c.pape@gmx.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution!
Add heuristics to allow to channel ais into being the first axis.
For better compatibility with preview data in napari, for example.