-
Notifications
You must be signed in to change notification settings - Fork 13
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
Export of segmentations to n5 doesn't use nearest neighbour interpolation #183
Comments
Sounds good, let's do both, modify the code you copied and also raise an issue in the bdv-core repo. |
There are now two PR pending fixing this:
If need be, we could think about copying the code to MoBIE until this is merged. |
Great - thanks @tischi! |
@tischi Would you be happy to copy this over to MoBIE for now? I should have time next week to make a project creator 2.0 with Matt's suggestions, and it'd be great to include this. |
@K-Meech Could you first poke Tobias about the PRs? That is, go into the PRs, ping him, and say that you are very interested as well and need this rather soon :) |
@tischi I'm going to start rewriting parts of the project creator soon. Could we copy this over at some point? |
I asked in gitter again, let's wait until tomorrow. |
@K-Meech Tobias will not look into it within the next 2 weeks => copy it over, I guess. |
@tischi I'm copying over this code now, and I noticed an issue. I added a quick fix here: K-Meech@84a9d1a |
This is fixed now. When the PRs above get merged, we can remove: ExportScalePyramid, DownsampleBlock and WriteSequenceToN5 from MoBIE |
ExportImagePlusAsN5Plugin in Fiji doesn't have options for the interpolation used during downsampling.
For segmentations, this results in values outside the labels for downsampled layers.
Would have to dig into this code https://github.com/bigdataviewer/bigdataviewer-core/blob/master/src/main/java/bdv/export/n5/WriteSequenceToN5.java / or put an issue on their repo.
The text was updated successfully, but these errors were encountered: