<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 9, 2016, at 5:59 PM, Cameron Shorter <<a href="mailto:cameron.shorter@gmail.com" class="">cameron.shorter@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type" class="">
  
  <div bgcolor="#FFFFFF" text="#000000" class="">
    -1 from me for including all python notebooks on the upcoming
    OSGeo-Live 9.5 release in a few days. I don't think we can make it
    ready in time.<br class="">
    <br class="">
    This is based on my preliminary review of the gsoc iso:<br class="">
<a class="moz-txt-link-freetext" href="http://osprey.ucdavis.edu/downloads/osgeo/gisvm/gisvm/9.5nightly/osgeo-live-nightly-gsoc-amd64-c83a060.iso">http://osprey.ucdavis.edu/downloads/osgeo/gisvm/gisvm/9.5nightly/osgeo-live-nightly-gsoc-amd64-c83a060.iso</a><br class=""></div></div></blockquote><div><br class=""></div><div>the are are several commit already after that iso</div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    I haven't done a comprehensive review, but have reviewed enough to
    feel the initial user experience when looking at notebooks is close,
    but is yet to meet our OSGeo-Live presentation standards.<br class="">
    <br class="">
    My comments so far:<br class="">
    <br class="">
    1. Quickstart review comments haven't been addressed yet, although
    hopefully Massimo should have that complete soon.<br class=""></div></div></blockquote><div><br class=""></div><div><br class=""></div><div>the quickstart and overview with your comments addressed:</div><div><br class=""></div><div><a href="https://github.com/epifanio/OSGeoLive-doc/blob/master/en/quickstart/jupyter_quickstart.rst" class="">https://github.com/epifanio/OSGeoLive-doc/blob/master/en/quickstart/jupyter_quickstart.rst</a></div><div><br class=""></div><div><a href="https://github.com/epifanio/OSGeoLive-doc/blob/master/en/overview/jupyter_overview.rst" class="">https://github.com/epifanio/OSGeoLive-doc/blob/master/en/overview/jupyter_overview.rst</a></div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    2. Structure of notebook docs directory should be by project, rather
    than singled out as a GSoC directory, then by project. Long term we
    want GSoC content to be part of the core OSGeo-Live offering.<br class=""></div></div></blockquote><div><br class=""></div><div>That’s a personal thought, you had 10 months to think about this.</div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    3. We are missing an obvious Table of Contents.<br class=""></div></div></blockquote><div><br class=""></div><div><br class=""></div><div>it is here:</div><div><br class=""></div><div><a href="https://github.com/epifanio/OSGeoLive-Notebooks/blob/osgeolive/index.ipynb" class="">https://github.com/epifanio/OSGeoLive-Notebooks/blob/osgeolive/index.ipynb</a></div><div><br class=""></div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    4. I've randomly picked iris_basics.ipynb, running step 2, I get a
    warning about a library being deprecated. (It still works, but this
    is the sort of thing that should be cleaned up before presenting to
    a new user).<br class=""></div></div></blockquote><div><br class=""></div><div>i’m not contributing to this, but it is just a warning.</div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    5. There should be an introductory notebook which starts with "Press
    the RUN CELL button to see the output of [In] cells”.<br class=""></div></div></blockquote><div><br class=""></div><div>it is here:</div><div><br class=""></div><div><a href="https://github.com/epifanio/OSGeoLive-Notebooks/blob/bf2fcb56480738796cf8a0e0a1a1fc5195f96e92/introduction-to-jupyter-notebook.ipynb" class="">https://github.com/epifanio/OSGeoLive-Notebooks/blob/bf2fcb56480738796cf8a0e0a1a1fc5195f96e92/introduction-to-jupyter-notebook.ipynb</a></div><div><br class=""></div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    6.1 I've found a number of spelling and grammar errors in the first
    GSoC notebook I reviewed. Nothing major, but they should be
    identified and cleaned up all the same.<br class="">
    For example, this is the first paragraph of the GDAL/OGR Quickstart
    (which is the first Docbook I looked at):<br class="">
    <br class="">
    <i class="">"The first Notebook is dedicared [SPELLING] to the use of the
      Geospatial Data Abstraction Library GDAL [USE BRACES AROUND FIRST
      USE OF CAPITALS] from the bash command line. GDAL is a powerful
      translator library for raster and vector geospatial data formats
      [INSERT FULL STOP] it presents a single raster abstract data model
      and vector abstract data model to the calling application for all
      supported formats.”</i></div></div></blockquote><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    6.2 In the same notebook, I found:<br class="">
    <i class="">"#unfinished - need to add ogr2ogr features yet, I'm considering
      to split this notebook in 2 parts (one for gdal and one for
      ogr2ogr) i'll study the gdal tutorial released recently"</i><br class="">
    <br class="">
    Massimo notes that he is fixing this, but my point is that I'm
    assuming that if reviewed, similar issues would be found in other
    notebooks.<br class=""></div></div></blockquote><div><br class=""></div><div>they are fixed:</div><div><br class=""></div><div><a href="https://github.com/epifanio/OSGeoLive-Notebooks/blob/osgeolive/GSoC-2015/Access to Geospatial data/GDAL-OGR Quickstart.ipynb" class="">https://github.com/epifanio/OSGeoLive-Notebooks/blob/osgeolive/GSoC-2015/Access%20to%20Geospatial%20data/GDAL-OGR%20Quickstart.ipynb</a></div><div><br class=""></div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    7. The term "IPython" and "Jupyter" is used interchangeably, which
    would be confusing for a new user, and I'd be interested to
    understand if is technically incorrect?<br class="">
    <br class=""></div></div></blockquote><div><br class=""></div><div>where? </div><div>if you refer to the the code, that’s normal ipython is part of jupyter.</div><br class=""><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    8. Quantity of material to review and short time period. We have
    days left before the our final release candidate is due, and
    multiple notebooks (say 20+ ?) which I suggest need review to get to
    same level of review as Project Overviews and Quoickstarts.<br class=""></div></div></blockquote><div><br class=""></div><div><br class=""></div><div>the GSoC notebooks have been reviewed, </div><div>you’re finding typos ok, my apology … as a non native speaker</div><div>but again, you had 10 months to think about this</div><div><br class=""></div><div>if you find more, i’m open to fix them ASAP</div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    9. Also on quantity of material, we are introducing a maintenance
    load for future OSGeo-Live releases, and we are as yet to build up a
    community of volunteers willing to maintain these notebooks. Massimo
    and Brian have been doing a great job creating these notebooks, but
    are already under time pressure to update for this release, let
    alone future releases. I think we need to reach out to all projects
    with the offer of:<br class="">
    "A notebook has been created for your project. If you are prepared
    to review it now, and commit to verifying it is still valid before
    each OSGeo-Live release, then we will include the notebook on
    OSGeo-Live.”</div></div></blockquote><div><br class=""></div><div>As per the GSoC notebooks, I think I’m demonstrating that you should not worry about their maintenance.</div><div><br class=""></div><div>I addressed your comment, I can’t speak for the notebooks under the projects directory.</div><div><br class=""></div><div>My recommendation is:</div><div><br class=""></div><div>we can rename the GSoC-2015 directory .. perhaps with the name of the idea itself which is:</div><div><br class=""></div><div>Open Source Geospatial Notebooks.</div><div><br class=""></div><div>then if you have concerns about the “projects" directory, we can comment it out from the table of contents and leave it on disc.</div><div><br class=""></div><div>note the quick start and overview they both uses the GSoC material, so they will not be affected.</div><div><br class=""></div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div bgcolor="#FFFFFF" text="#000000" class="">
    <br class="">
    --<br class="">
    So my recommendation is to either:<br class="">
    A. Include Notebooks on next OSGeo-Live 9.5, but remove reference to
    it from main docs. (My preference at this point)<br class="">
    <br class="">
    B. Include ONE Jupyter notebook example which can be found from main
    docs, which we make sure is well reviewed. All the remainder of
    Notebooks to be hidden. (This will likely be a messy solution, as
    we'd probably need to move directories around so they are hard to
    find for a new user).<br class="">
    <br class="">
    <div class="moz-cite-prefix">On 9/03/2016 11:44 pm, massimo di
      stefano wrote:<br class="">
    </div>
    <blockquote cite="mid:6D6AF33D-08CD-4B0E-8E2D-9E17A88AC386@gmail.com" type="cite" class="">
      <div class="">To All,</div>
      <div class=""><br class="">
      </div>
      <div class="">during the last 2 years I worked in having the
        python software ecosystem included in the OSGeo-Live.</div>
      <div class="">The PR proposed below [1] is a first step towards
        the official inclusion of this work into the project.</div>
      <div class=""><br class="">
      </div>
      <div class="">As you may have noticed from previous e-email [2]
        there isn’t a clear agreement on the decision to take.</div>
      <div class="">We had an IRC discussion (logs [3]) with an initial
        motion to merge the notebook PR.</div>
      <div class=""><br class="">
      </div>
      <div class=""><span style="color: rgb(38, 50, 56); font-family:
          arial, sans-serif; font-size: 13px; widows: 1;" class="">I
          would like to have a voting started here on the mailing.</span></div>
      <div class="">this is mine:</div>
      <div class=""><br class="">
      </div>
      <div class=""> +1</div>
      <div class=""><br class="">
      </div>
      <div class=""><br class="">
      </div>
      <div class="">Thanks,</div>
      <div class="">Massimo.</div>
      <div class=""><br class="">
      </div>
      <div class=""><br class="">
      </div>
      <div class="">===</div>
      <div class=""><br class="">
      </div>
      [1] <a moz-do-not-send="true" href="https://github.com/OSGeo/OSGeoLive-Notebooks/pull/5" class="">https://github.com/OSGeo/OSGeoLive-Notebooks/pull/5</a>
      <div class="">[2] <a moz-do-not-send="true" href="http://lists.osgeo.org/pipermail/live-demo/2016-March/011010.html" class="">http://lists.osgeo.org/pipermail/live-demo/2016-March/011010.html</a></div>
      <div class="">[3] <a moz-do-not-send="true" href="http://irclogs.geoapt.com/osgeolive/#osgeolive.2016-03-07.log" class="">http://irclogs.geoapt.com/osgeolive/%23osgeolive.2016-03-07.log</a></div>
      <br class="">
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br class="">
      <pre wrap="" class="">_______________________________________________
Live-demo mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Live-demo@lists.osgeo.org">Live-demo@lists.osgeo.org</a>
<a class="moz-txt-link-freetext" href="http://lists.osgeo.org/mailman/listinfo/live-demo">http://lists.osgeo.org/mailman/listinfo/live-demo</a>
<a class="moz-txt-link-freetext" href="http://live.osgeo.org/">http://live.osgeo.org</a>
<a class="moz-txt-link-freetext" href="http://wiki.osgeo.org/wiki/Live_GIS_Disc">http://wiki.osgeo.org/wiki/Live_GIS_Disc</a></pre>
    </blockquote>
    <br class="">
    <pre class="moz-signature" cols="72">-- 
Cameron Shorter,
Software and Data Solutions Manager
LISAsoft
Suite 112, Jones Bay Wharf,
26 - 32 Pirrama Rd, Pyrmont NSW 2009

P +61 2 9009 5000,  W <a class="moz-txt-link-abbreviated" href="http://www.lisasoft.com/">www.lisasoft.com</a>,  F +61 2 9009 5099</pre>
  </div>

_______________________________________________<br class="">Live-demo mailing list<br class=""><a href="mailto:Live-demo@lists.osgeo.org" class="">Live-demo@lists.osgeo.org</a><br class="">http://lists.osgeo.org/mailman/listinfo/live-demo<br class="">http://live.osgeo.org<br class="">http://wiki.osgeo.org/wiki/Live_GIS_Disc</div></blockquote></div><br class=""></body></html>