schema: added asset_as_band property in render object - #20
Conversation
asset_as_band (bool): tell rio-tiler that each asset is a 1 band dataset, so expression Asset1/Asset2 can be passed. Refs: stac-extensions#12 Implements: https://learn.microsoft.com/en-us/azure/planetary-computer/render-configuration?tabs=restapi#1-expressions Implements: https://developmentseed.org/titiler/endpoints/stac/#tiles Co-authored-by: Fabrice Brito <fabrice.brito@terradue.com>
|
@emmanuelmathot cannot reproduce locally the error: Not sure what the error is about in the pipeline |
emmanuelmathot
left a comment
There was a problem hiding this comment.
Seems good to me but an example would help to understand how it works
Co-authored-by: Leonardo Mizzoni <leonardo.mizzoni@terradue.com> Refs: stac-extensions#12 Refs: stac-extensions#20 (review)
|
@emmanuelmathot requested example added |
emmanuelmathot
left a comment
There was a problem hiding this comment.
I suggest simplifying the examlple to focus on render
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
|
@sim13pods I cannot push on your branch the simplified example (you did not allow changes for reviewers). So that's why I proposed suggestions. Please apply them to have a simpler example., Then we are good to merge. |
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
emmanuelmathot
left a comment
There was a problem hiding this comment.
Please remove the histograms. They are huge and there is no value adding them here and may confuse adopters.
Histograms removed Co-authored-by: Leonardo Mizzoni <leonardo.mizzoni@terradue.com> Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com> Refs: stac-extensions#12 Refs: stac-extensions#12 Refs: stac-extensions#20 (review)
Refs: [stac-extensions#12](https://github.com/Terradue/render/issues/12) Implements: https://learn.microsoft.com/en-us/azure/planetary-computer/render-configuration?tabs=restapi[#1](https://github.com/Terradue/render/issues/1)-expressions Implements: https://developmentseed.org/titiler/endpoints/stac/#tiles Co-authored-by: Fabrice Brito <fabrice.brito@terradue.com>
|
@emmanuelmathot histograms removed (GitHub editor somehow stopped working) and CHANGELOG updated |
emmanuelmathot
left a comment
There was a problem hiding this comment.
@sim13pods please check the asset_as_band values in your example. You can test with npm test
Co-authored-by: Emmanuel Mathot <emmanuel.mathot@gmail.com>
asset_as_band (bool): tell rio-tiler that each asset is a 1 band dataset, so expression Asset1/Asset2 can be passed.
Resolves: #12
Implements: https://learn.microsoft.com/en-us/azure/planetary-computer/render-configuration?tabs=restapi#1-expressions
Implements: https://developmentseed.org/titiler/endpoints/stac/#tiles
Co-authored-by: Fabrice Brito fabrice.brito@terradue.com