<div dir="ltr">Hi Markus!<br><div class="gmail_extra"><br><div class="gmail_quote">2018-07-24 14:43 GMT+02:00 Markus Neteler <span dir="ltr"><<a href="mailto:neteler@osgeo.org" target="_blank">neteler@osgeo.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div>Hi Roberta,<span><br><br><div class="gmail_quote"><div dir="ltr">Roberta Fagandini <<a href="mailto:robifagandini@gmail.com" target="_blank">robifagandini@gmail.com</a>> schrieb am Di., 24. Juli 2018, 11:38:<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 dir="ltr">Hi Martin,<div>as Moritz suggested, Id like to have your opinion about the implementation of a new procedure for downloading Sentinel images avoiding too many dependencies.</div></div></blockquote></div></span></div><div dir="auto"><br></div><div dir="auto">I'm following loosely as I'm on the road at time but I'd like to better understand this discussion.</div><span><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I'm working on a module (i.sentinel.preproc [0]) which wraps the import and the atmospheric correction of sentinel images. My intention is to add also the download phase but I'd like to avoid dependencies. That's</div></div></blockquote></div></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div> the reason why I'm not using i.sentinel.download at the moment.</div></div></blockquote></div></div><div dir="auto"><br></div></span><div dir="auto">This isn't clear to me so far. The only potential dependency would be</div><div dir="auto"><br></div><div dir="auto">g.extension i.sentinel</div><div dir="auto"><br></div><div dir="auto">right? Which uses a pip installable library.</div></div></blockquote><div><br></div><div><div>i.sentinel.download requires both sentinelsat and pandas (both installable with pip).</div><div>In a previous discussion, I asked about the installation of the python library Py6S and the dev community strongly encouraged me to avoid too many dependencies (also Py6S requires pandas). Consequently I wrote my own routine to read AERONET file and compute AOT at 550nm. </div><div>That's the reason why I'm trying to avoid dependencies.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><span><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I found this python script [1] that maybe could solve the problem of dependencies and it could be a good starting point.</div></div></blockquote></div></div><div dir="auto"><br></div></span><div dir="auto">For me that script doesn't work, tried a few days ago (while i.sentinel.download does work).</div><div dir="auto">What is the advantage of having the dependency [1] over an existing GRASS add-on?</div></div></blockquote><div><br></div><div>I have not tested the script yet, but looking at the code I think I can try to implement part of the code in my module changing it accordingly with the requirements of a GRASS GIS module. This should avoid the installation of both sentinelsat and pandas. Right?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><span><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>What do you think?</div><div><br></div><div>As always any feedback is welcome!</div></div></blockquote></div></div><div dir="auto"><br></div></span><div dir="auto">I'd rather consider to share code with the existing addon than starting from scratch.</div><div dir="auto">But perhaps I'm still overlooking something.</div></div></blockquote><div><br></div><div>Obviously, it would be easier for me to use the existing <span style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">addon (i.sentinel.download) but I struggled quite a lot to avoid pandas and other dependencies<span style="font-size:12.8px;text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline"> ;-).</span> </span>I do not know if it makes sense to add it now in order to integrate a module that already exists and that users can use separately. What do you think?</div><div><br></div><div><br></div><div>Roberta</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="auto"><br></div><div dir="auto">My 0.02 holiday cents :)</div><span class="gmail-m_-2104075428020883539gmail-m_-8844587878793373901gmail-m_312723930079646587gmail-m_9035031871222480508gmail-m_-7245984482761430135m_-7382082853245335337HOEnZb"><font color="#888888"><div dir="auto">Markus</div></font></span><span><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>Thanks in advance.<br><div class="gmail_extra"><br></div><div class="gmail_extra">Roberta</div><div class="gmail_extra"><br></div><div class="gmail_extra">[0] <a href="https://github.com/RobiFag/GRASS_clouds_and_shadows" rel="noreferrer" target="_blank">https://github.com/RobiFag<wbr>/GRASS_clouds_and_shadows</a></div><div class="gmail_extra">[1] <a href="https://github.com/olivierhagolle/Sentinel-download" rel="noreferrer" target="_blank">https://github.com/olivier<wbr>hagolle/Sentinel-download</a></div><div class="gmail_extra"><div class="gmail_quote"><br></div></div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div class="gmail_extra"><div class="gmail_quote"><br></div></div></div></div></blockquote></div></div></span></div>
</blockquote></div><br></div></div>