<div><div><div dir="auto">Thanks everyone, my questions are answered (and the fast response is great). </div></div><div dir="auto"><br></div><div dir="auto">Next step is to make a motion (providing  two weeks for committee members to respond since this is an email based committee). </div><div dir="auto"><br></div><div dir="auto">Jo you took part in this discussion - can I ask you to  make a motion please? </div></div><div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 4, 2019 at 6:03 AM Angelos Tzotsos <<a href="mailto:gcpp.kalxas@gmail.com" target="_blank">gcpp.kalxas@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jody and thanks for the feedback.<br>
<br>
Comments inline:<br>
<br>
On 6/3/19 10:53 PM, Jody Garnett wrote:<br>
> First up welcome pygeoapi and thanks for your application.<br>
><br>
> The first thing I checked was if the project has an osgeo project page (it<br>
> does <a href="https://www.osgeo.org/projects/pygeoapi/" rel="noreferrer" target="_blank">https://www.osgeo.org/projects/pygeoapi/</a>). I think the "emerging suite<br>
> of OGC API standards" has a name of some sort but I agree it has not been<br>
> fixed yet?  I like that there is a demo server :)<br>
<br>
The standard name was switched from WFS3 to OGC API - Features but lets <br>
wait for the OGC Hackathon to be over to make sure another change does <br>
not happen. We will update the page once we have feedback.<br>
<br>
><br>
> 1. Geospatial<br>
><br>
> README.md <<a href="https://github.com/geopython/pygeoapi/blob/master/README.md" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/blob/master/README.md</a>>,<br>
> looks good :) As above you may wish to name the standard being implemented.<br>
<br>
Will be added as above<br>
<br>
><br>
> 2. Open Source license [6]<br>
><br>
> LICENSE <<a href="https://github.com/geopython/pygeoapi/blob/master/LICENSE" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/blob/master/LICENSE</a>> is<br>
> good, you may wish to use markdown version here<br>
> <<a href="https://github.com/IQAndreas/markdown-licenses" rel="noreferrer" target="_blank">https://github.com/IQAndreas/markdown-licenses</a>> for pretty?<br>
> For community project (rather than just website) we ask that headers also<br>
> be checked... I see you are doing that below.<br>
<br>
Fixed in <a href="https://github.com/geopython/pygeoapi/pull/165" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/pull/165</a><br>
<br>
><br>
><br>
><br>
> 3. Open to contributions [7]<br>
><br>
> Nice to see evidence of contributors<br>
> <<a href="https://github.com/geopython/pygeoapi/graphs/contributors" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/graphs/contributors</a>> taking part<br>
> (also noted diverse authors in headers).<br>
><br>
> It is a good practice, that we check for to at least recommend, to use<br>
> CONTRIBUTING.md. This outlines what you expect of contributors and is a<br>
> good spot to remind folks they are agreeing to publish their work using<br>
> that MIT license.<br>
<br>
Fixed in <a href="https://github.com/geopython/pygeoapi/pull/164" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/pull/164</a><br>
<br>
><br>
> 4. License headers [8]<br>
><br>
> Doing a few spot checks:<br>
> - pygeoapi-serverless-config.yml<br>
> <<a href="https://github.com/geopython/pygeoapi/blob/master/aws-lambda/pygeoapi-serverless-config.yml" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/blob/master/aws-lambda/pygeoapi-serverless-config.yml</a>><br>
> -<br>
> needs header<br>
> - pygeoapi/debian/copyright<br>
> <<a href="https://github.com/geopython/pygeoapi/blob/master/debian/copyright" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/blob/master/debian/copyright</a>> -<br>
> actually has a different license<br>
> - pygeoapi/docker/entrypoint.sh - needs header<br>
> - pygeoapi/docker/examples - no sign of header?<br>
> - pygeoapi/api.py<br>
> <<a href="https://github.com/geopython/pygeoapi/blob/master/pygeoapi/api.py" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/blob/master/pygeoapi/api.py</a>> - core<br>
> code looks good<br>
> - tests/data/README.md - nicely done documenting where sample data was<br>
> obtained from :)<br>
<br>
Fixed in <a href="https://github.com/geopython/pygeoapi/pull/165" rel="noreferrer" target="_blank">https://github.com/geopython/pygeoapi/pull/165</a><br>
<br>
><br>
> Let me know about the feedback above, and thanks again for your application<br>
<br>
Best,<br>
Angelos<br>
<br>
<br>
-- <br>
Angelos Tzotsos, PhD<br>
Charter Member<br>
Open Source Geospatial Foundation<br>
<a href="http://users.ntua.gr/tzotsos" rel="noreferrer" target="_blank">http://users.ntua.gr/tzotsos</a><br>
<br>
</blockquote></div></div>
</div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>--</div><div>Jody Garnett</div></div></div></div></div>