-
Notifications
You must be signed in to change notification settings - Fork 409
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 support to multi-band color mapping for COG layers #9588
Comments
As part of fetching metadata, we read only one image to get metadata info of the tif source. Reading raster data of the entire image can puts too much load on CPU and can severly impact performance, hence we need to specify a So my thoughts, we can get the banding information from the user in catalog and add it to COG source for user defined rendering of tif source and skip any reading of banding info from the image in catalog. Mocks
We can discuss on this @offtherailz to establish an acceptance criteria |
Updated acceptance criteria
|
Max/MinAbout min/max please ask to @tdipisa if it should be done in this dev cycle. I was thinking to work values of data more with advanced style features (like calculations an so and so) in a context of an advanced raster styler). Apha channelYes, good idea. I think it can stay always together with the RGB, and optional (can be unselected). look at similar tools to get an idea of how it should work. |
Since the support for min/max in 'layer settings->style' is included in your evaluation please consider also this in the development so that we will be able to better support also these special cases if needed. |
blocked by #10083 |
… layers (geosolutions-it#9857) (cherry picked from commit d683b82)
Description
As visible here, we can associate the colors to bands. Actually mapstore supports only standard 3 bands RGB or 1 band gray (because it is the openlayers default)
We have to implement the support to associate colors to bands.
This means:
What kind of improvement you want to add? (check one with "x", remove the others)
Other useful information
The text was updated successfully, but these errors were encountered: