Skip to content

Commit c9a480a

Browse files
Merge pull request #20 from developmentseed/patch/parse-and-format-rescales
parse and format rescale array
2 parents 90b849f + 125b2c5 commit c9a480a

File tree

2 files changed

+43
-5
lines changed

2 files changed

+43
-5
lines changed

tests/test_dependency_params.py

+30
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
"""test get_dependency_params."""
2+
3+
from titiler.core.dependencies import RescalingParams
4+
from titiler.stacapi.factory import get_dependency_params
5+
6+
7+
def test_get_params_rescale():
8+
"""test get_dependency_params for rescale."""
9+
10+
def _parse_rescale(rescale):
11+
rescales = []
12+
for r in qs["rescale"]:
13+
if not isinstance(r, str):
14+
rescales.append(",".join(map(str, r)))
15+
else:
16+
rescales.append(r)
17+
18+
return rescales
19+
20+
qs = {"rescale": ["0,1", "2,3"]}
21+
assert get_dependency_params(
22+
dependency=RescalingParams,
23+
query_params={"rescale": _parse_rescale(qs)},
24+
) == [(0.0, 1.0), (2.0, 3.0)]
25+
26+
qs = {"rescale": [[0, 1], [2, 3]]}
27+
assert get_dependency_params(
28+
dependency=RescalingParams,
29+
query_params={"rescale": _parse_rescale(qs)},
30+
) == [(0.0, 1.0), (2.0, 3.0)]

titiler/stacapi/factory.py

+13-5
Original file line numberDiff line numberDiff line change
@@ -810,11 +810,19 @@ def get_tile( # noqa: C901
810810
):
811811
image = post_process(image)
812812

813-
if rescale := get_dependency_params(
814-
dependency=self.rescale_dependency,
815-
query_params=query_params,
816-
):
817-
image.rescale(rescale)
813+
if "rescale" in query_params:
814+
rescales = []
815+
for r in query_params["rescale"]:
816+
if not isinstance(r, str):
817+
rescales.append(",".join(map(str, r)))
818+
else:
819+
rescales.append(r)
820+
821+
if rescale := get_dependency_params(
822+
dependency=self.rescale_dependency,
823+
query_params={"rescale": rescales},
824+
):
825+
image.rescale(rescale)
818826

819827
if color_formula := get_dependency_params(
820828
dependency=self.color_formula_dependency,

0 commit comments

Comments
 (0)