[Geoserver-devel] GSIP-138 (vector tiles community module to extension)

davidblasby · October 18, 2016, 6:02pm

Hi,

I did some work in the VT community module;

* very very minor code clean up (formatting)
* integrated one of my old PRs (degenerate geometries filtering)
* added some documentation pages

I believe it's now ready to move to an extension - please review/vote.

Thanks a lot,
David

Bart_van_den_Eijnde1 · October 18, 2016, 9:58pm

+1 to promote gs-vectortiles to extension. Thanks, Dave, for your efforts on this one. I changed the title to include "Call for Vote" to attract attention because you called for a vote in your text below.

The most recent Cobertura release does not support Java 8 bytecode, and Cobertura master (2.1.2-SNAPSHOT) does not build from source for me under OpenJDK 8 (and that is as far down that rabbit hole that I am prepared to go).

EclEmma (Eclipse coverage plugin) reports a quite respectable code coverage of 71.3% for gs-vectortiles.

Kind regards,
Ben.

On 19/10/16 07:02, Dave Blasby wrote:

Hi,

I did some work in the VT community module;

* very very minor code clean up (formatting)
* integrated one of my old PRs (degenerate geometries filtering)
* added some documentation pages

https://github.com/geoserver/geoserver/wiki/GSIP-138

I believe it's now ready to move to an extension - please review/vote.

Thanks a lot,
David

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/>
New Zealand

jive · October 19, 2016, 5:13am

Thanks Dave I look forward to reviewing this.

···

On 18 October 2016 at 13:02, Dave Blasby <dblasby@anonymised.com> wrote:

Hi,

I did some work in the VT community module;

very very minor code clean up (formatting)

integrated one of my old PRs (degenerate geometries filtering)

added some documentation pages

https://github.com/geoserver/geoserver/wiki/GSIP-138

I believe it’s now ready to move to an extension - please review/vote.

Thanks a lot,
David

Check out the vibrant tech community on one of the world’s most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot

Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

–
Jody Garnett

ianturton · October 19, 2016, 7:50am

+1

Ian

···

On 18 October 2016 at 22:58, Ben Caradoc-Davies <ben@anonymised.com> wrote:

+1 to promote gs-vectortiles to extension. Thanks, Dave, for your
efforts on this one. I changed the title to include “Call for Vote” to
attract attention because you called for a vote in your text below.

The most recent Cobertura release does not support Java 8 bytecode, and
Cobertura master (2.1.2-SNAPSHOT) does not build from source for me
under OpenJDK 8 (and that is as far down that rabbit hole that I am
prepared to go).

EclEmma (Eclipse coverage plugin) reports a quite respectable code
coverage of 71.3% for gs-vectortiles.

Kind regards,
Ben.

On 19/10/16 07:02, Dave Blasby wrote:

Hi,

I did some work in the VT community module;

very very minor code clean up (formatting)

integrated one of my old PRs (degenerate geometries filtering)

added some documentation pages

https://github.com/geoserver/geoserver/wiki/GSIP-138

I believe it’s now ready to move to an extension - please review/vote.

Thanks a lot,
David

Check out the vibrant tech community on one of the world’s most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot

Geoserver-devel mailing list
Geoserver-devel@anonymised.comsourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

–
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/>
New Zealand

Check out the vibrant tech community on one of the world’s most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot

Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

–

Ian Turton

Kevin_Smith1 · October 19, 2016, 8:47pm

+1

On 2016-10-19 12:50 AM, Ian Turton wrote:

+1

Ian

On 18 October 2016 at 22:58, Ben Caradoc-Davies <ben@anonymised.com
<mailto:ben@anonymised.com>> wrote:

    +1 to promote gs-vectortiles to extension. Thanks, Dave, for your
    efforts on this one. I changed the title to include "Call for Vote" to
    attract attention because you called for a vote in your text below.

    The most recent Cobertura release does not support Java 8
    bytecode, and
    Cobertura master (2.1.2-SNAPSHOT) does not build from source for me
    under OpenJDK 8 (and that is as far down that rabbit hole that I am
    prepared to go).

    EclEmma (Eclipse coverage plugin) reports a quite respectable code
    coverage of 71.3% for gs-vectortiles.

    Kind regards,
    Ben.

    On 19/10/16 07:02, Dave Blasby wrote:
    > Hi,
    >
    > I did some work in the VT community module;
    >
    > * very very minor code clean up (formatting)
    > * integrated one of my old PRs (degenerate geometries filtering)
    > * added some documentation pages
    >
    > https://github.com/geoserver/geoserver/wiki/GSIP-138
    <https://github.com/geoserver/geoserver/wiki/GSIP-138>
    >
    > I believe it's now ready to move to an extension - please
    review/vote.
    >
    > Thanks a lot,
    > David
    >
    >
    ------------------------------------------------------------------------------
    > Check out the vibrant tech community on one of the world's most
    > engaging tech sites, SlashDot.org! http://sdm.link/slashdot
    > _______________________________________________
    > Geoserver-devel mailing list
    > Geoserver-devel@lists.sourceforge.net
    <mailto:Geoserver-devel@lists.sourceforge.net>
    > https://lists.sourceforge.net/lists/listinfo/geoserver-devel
    <https://lists.sourceforge.net/lists/listinfo/geoserver-devel>
    >

    --
    Ben Caradoc-Davies <ben@anonymised.com <mailto:ben@anonymised.com>>
    Director
    Transient Software Limited <http://transient.nz/>
    New Zealand

    ------------------------------------------------------------------------------
    Check out the vibrant tech community on one of the world's most
    engaging tech sites, SlashDot.org! http://sdm.link/slashdot
    _______________________________________________
    Geoserver-devel mailing list
    Geoserver-devel@lists.sourceforge.net
    <mailto:Geoserver-devel@lists.sourceforge.net>
    https://lists.sourceforge.net/lists/listinfo/geoserver-devel
    <https://lists.sourceforge.net/lists/listinfo/geoserver-devel>

--
Ian Turton

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot

_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

--
Kevin Michael Smith
<smithkm@anonymised.com>