<div dir="ltr"><div><div>Please don't change the default behavior for 3-band rasters to min/max; while it does make in many scenarios for 1-band rasters (DEM, precipitation value, vegetation index, etc.), this is a terrible default value for most - if not all - satellite imagery products (i.e. Sentinel-2, LANDSAT-{5,7,8}, ASTER, etc.).<br><br></div>IMO, we need to have two separate default option: one for 3-band raster min/max (which doesn't change), and one for 1-band raster (which is changed to min/max).<br><br></div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 2, 2016 at 7:00 PM, Nyall Dawson <span dir="ltr"><<a href="mailto:nyall.dawson@gmail.com" target="_blank">nyall.dawson@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2 September 2016 at 21:50, G. Allegri <<a href="mailto:giohappy@gmail.com">giohappy@gmail.com</a>> wrote:<br>
> +1<br>
><br>
> I would also make min/max the default option, because I've has many users<br>
> (mostly newcomers) were struggling too find out why some pixels were not<br>
> displaying as expected. It was because the statistics weren't including<br>
> some, few, important pixels.<br>
<br>
</span>Yes please! There's so many users on stackexchange who get confused<br>
with this same issue.<br>
<span class="HOEnZb"><font color="#888888"><br>
Nyall<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> Anyway I know that, generally, cutting the tails provide a better visual<br>
> result.<br>
><br>
> giovanni<br>
><br>
><br>
> Il 02/set/2016 12:39, "Even Rouault" <<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>> ha<br>
> scritto:<br>
>><br>
>> Hi,<br>
>><br>
>> There has been interest expressed to improve the usability of the contrast<br>
>> enhancement function. Basically there are workflows where you want the<br>
>> min/max<br>
>> values used by contrast enhancement to be updated each time you pan or<br>
>> zoom<br>
>> the canvas. From what I've seen, here's what currently exists:<br>
>> - a long method (6 clicks): through the Style tab of the layer properties<br>
>> - a faster method (2 clicks): through the "Strech using current extent" of<br>
>> the<br>
>> contextual menu of the layer panel.<br>
>><br>
>> When you have several rasters loaded, even the 2-click method is<br>
>> inconvenient,<br>
>> so there's a need for a 0-click solution (once the layers have been<br>
>> configured)<br>
>><br>
>> Another shortcoming in the current implementation is that the settings of<br>
>> the<br>
>> "Load min/max values" foldable group are not persistant, which requires<br>
>> re-<br>
>> setting them if you're not happy with the default Cumlative count cut<br>
>> method.<br>
>> And if you use the "Strech using current extent" menu item, the genuine<br>
>> min/max values are used (not the 2-98% cumulative count cut)<br>
>><br>
>> So I was thinking to something along the following lines:<br>
>> - make the min/max settings persistant<br>
>> - remove the "Load" button and "Clip extent to canvas" checkbox, and<br>
>> replace<br>
>> them by 3 radio buttons to determine the scope of statistics : "Whole<br>
>> raster",<br>
>> "Current canvas" and "Updated canvas". See the attached<br>
>> proposal_min_max.png.<br>
>> The function of the Load button would be replaced by the general Apply /<br>
>> OK<br>
>> buttons.<br>
>> - when the user manually enters the min/max values, the 2 groups of radio<br>
>> buttons (method to compute min/max and scope of statistics) would be<br>
>> unchecked<br>
>> so that is is clear that they don't come from computed statistics. I also<br>
>> think that when you select "Current canvas", once the Apply/OK buttons<br>
>> have<br>
>> been pushed, the checked state of Current canvas shouldn't be saved. This<br>
>> way<br>
>> when you display the layer properties you have a good idea of where the<br>
>> current min/max values come from.<br>
>> - make the "Strech using current extent" method honour the way min/max are<br>
>> computed in the min/max settings instead of using systematically the<br>
>> min/max.<br>
>> So when "Updated canvas" would be selected, it would have no (extra)<br>
>> effect as<br>
>> expected.<br>
>><br>
>> Any opinions ?<br>
>><br>
>> Even<br>
>><br>
>> --<br>
>> Spatialys - Geospatial professional services<br>
>> <a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
>><br>
>> ______________________________<wbr>_________________<br>
>> Qgis-developer mailing list<br>
>> <a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
>> List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
>> Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
><br>
><br>
> ______________________________<wbr>_________________<br>
> Qgis-developer mailing list<br>
> <a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
> List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
> Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
______________________________<wbr>_________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
List info: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a><br>
Unsubscribe: <a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" rel="noreferrer" target="_blank">http://lists.osgeo.org/<wbr>mailman/listinfo/qgis-<wbr>developer</a></div></div></blockquote></div><br></div>