<div dir="ltr"><div dir="ltr">Hi Stefan,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El lun., 2 dic. 2019 a las 11:45, Stefan Blumentrath (<<a href="mailto:Stefan.Blumentrath@nina.no">Stefan.Blumentrath@nina.no</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="NO-BOK">
<div class="gmail-m_376873077030647046WordSection1">
<p class="MsoNormal">Dear all,<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><span lang="EN-GB">Recently, I have been working with i.sentinel addons and propose some fixes / enhancements in the PR below:<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><a href="https://github.com/OSGeo/grass-addons/pull/68/files" target="_blank">https://github.com/OSGeo/grass-addons/pull/68/files</a><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Any objections against merging those changes?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">I would be very happy if someone could test!<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">In principle I would like to implement more changes later on like:<u></u><u></u></span></p>
<ul style="margin-top:0cm" type="disc">
<li class="gmail-m_376873077030647046MsoListParagraph" style="margin-left:0cm"><span lang="EN-GB">Using vsizip (in i.sentinel.import) and avoid extracting data from zip archives (at least offering it as a flag<u></u><u></u></span></li><li class="gmail-m_376873077030647046MsoListParagraph" style="margin-left:0cm"><span lang="EN-GB">Parse all metadata in i.sentinel.import and pass them to the subsequent modules (i.sentinel.preproc + i.sentinel.mask) so the latter do not have to do it again (and
 thus can be simplified)</span></li></ul></div></div></blockquote><div>+1 for these 2 :)<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="NO-BOK"><div class="gmail-m_376873077030647046WordSection1"><br><ul style="margin-top:0cm" type="disc"><li class="gmail-m_376873077030647046MsoListParagraph" style="margin-left:0cm"><span lang="EN-GB">Include i.sentinel.mask in i.sentinel.preproc and produce t.register output (like i.sentinel.import does)</span></li></ul></div></div></blockquote><div>I'm not so sure about the merging (+1 for the t.register output though :)). <br></div><div>What if someone does not want to get clouds masked? Will it be optional then, as ON/OFF flag to mask clouds and shadows? Will you anyway keep the i.sentinel.mask add-on?<br></div><div><br></div><div>my 0.02 cents</div><div>Vero<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="NO-BOK"><div class="gmail-m_376873077030647046WordSection1">
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Any thoughts?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Cheers<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Stefan<u></u><u></u></span></p>
</div>
</div>

_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org" target="_blank">grass-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/grass-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/grass-dev</a></blockquote></div></div>