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

Simplifying multiscale coordinateTransformations #201

Closed
clbarnes opened this issue Jul 5, 2023 · 1 comment
Closed

Simplifying multiscale coordinateTransformations #201

clbarnes opened this issue Jul 5, 2023 · 1 comment

Comments

@clbarnes
Copy link

clbarnes commented Jul 5, 2023

Currently, in order to get a mapping from data to world coordinates for a given scale level, you need to look at up to 3 different coordinateTransformations fields: one on the single-scale of interest, one on the first single-scale of the datasets array (unless I've misunderstood [1]), and then the multiscales-level coordinate transformations. Including the multiscales-level transformation may reduce some duplication, but it means that scale level coordinateTransforms do not map from data to world coordinates. The deduplication is conditional, too - order constraints on coordinate transformations mean that if you have a scale in the multiscale-level transforms, you can't have any translations in the scale level transforms, so if you had those transformations (common in half-pixel offsets for downscaling), you'd need to duplicate the top-level scale anyway.

Would it be simpler to say "the coordinateTransformations field on each single-scale object describes the mapping from data to world coordinates", by getting rid of A) any interdependency between different scale levels and B) the multiscale-level transforms? As the transformations themselves become more complicated (e.g. #138 , which is at least explicit about the interdependency), simplifying the lookup and chaining of transforms in different places might be nice.

1: "If scaling information is not available or applicable for one of the axes, the value MUST express the scaling factor between the current resolution and the first resolution for the given axis, defaulting to 1.0 if there is no downsampling along the axis."

@clbarnes
Copy link
Author

clbarnes commented Jul 5, 2023

This seems to be discussed in #172

@clbarnes clbarnes closed this as completed Jul 5, 2023
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

No branches or pull requests

1 participant