<br><br><div class="gmail_quote">On Fri, Oct 5, 2012 at 12:56 PM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@mines-paris.org" target="_blank">even.rouault@mines-paris.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">> >> 3) why is AVERAGE_MP considered unfit for use in the gdaladdo docs?</div><div class="im">
> >> and why is it still available?<br>
> ><br>
> > Historical remain ? Could likely be undocumented.<br>
><br>
> ok I will remove it, should it be removed/commented from the code also?<br>
<br>
</div>Undocumenting seems appropriate. As far as removing the implementation, you<br>
might want to check with the original committer (FrankW)<br></blockquote><div><br></div><div>Etienne, </div><div><br></div><div>Are AVERAGE_MP and AVERAGE_MAGPHASE really different things?  That seems odd. </div><div><br>
</div><div>I don't know why it reports them as not ready for production use.  I would not want them removed, but I don't mind them not being documented.</div><div><br></div><div>I peeked at the qgis bug report about magphase but I see the test data is no longer available.  If there are problems with magphase support that can be demonstrated and filed as a bug I'd be interested in resolving them.</div>
<div><br></div><div>Best regards,</div></div>-- <br>---------------------------------------+--------------------------------------<br>I set the clouds in motion - turn up   | Frank Warmerdam, <a href="mailto:warmerdam@pobox.com" target="_blank">warmerdam@pobox.com</a><br>
light and sound - activate the windows | <a href="http://pobox.com/~warmerdam" target="_blank">http://pobox.com/~warmerdam</a><br>and watch the world go round - Rush    | Geospatial Software Developer<br><br>