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

Add initial API for downsampling images #35

Merged
merged 2 commits into from
Nov 12, 2018

Conversation

melissalinkert
Copy link
Member

loci.common.image.IImageScaler is the interface, which only contains one method. loci.common.image.SimpleImageScaler is the initial implementation backported with minor modifications from private PRs.

There is also a new unit test class (SimpleImageScalerTest), which should be passing.

Discussed in Formats meeting on 10 October; see also https://trello.com/c/spl6WleR/74-ome-tiff-pyramid-testing-phase-ii. This is a prerequisite for being able to do something like bfconvert -pyramid.

loci.common.image.IImageScaler is the interface, which only contains one
method.  loci.common.image.SimpleImageScaler is the initial
implementation backported with minor modifications from private PRs.
@joshmoore
Copy link
Member

.SimpleImageScaler.java.swp slipped in to your commit.

@sbesson
Copy link
Member

sbesson commented Oct 31, 2018

As discussed today, having this in an ome-common:6.0.0-m1 milestone would be beneficial so that the downsampling code can be consumed by Bio-Formats PR. Will list for a round of code review tomorrow.

@sbesson sbesson added this to the 6.0.0 milestone Oct 31, 2018
@sbesson
Copy link
Member

sbesson commented Nov 7, 2018

LGTM so that the interface and the simple implementation can be consumed by the bfconvert utility.

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.

3 participants