[Geoserver-devel] GSIP 110 - Promote Printing community module to extension

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


On Thu, Mar 27, 2014 at 9:27 AM, Mauro Bartolomeoli <
mauro.bartolomeoli@anonymised.com> wrote:

Please, vote here:

http://geoserver.org/display/GEOS/GSIP+110+-+Promoting+the+Printing+community+module+to+extension

+1, about time!

Cheers
Andrea

--

Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------

+1

···

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK for more information.

Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information.

Ing. Alessio Fabiani
@alfa7691
Founder/Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 331 6233686

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


On Thu, Mar 27, 2014 at 9:36 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:



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

On Thu, Mar 27, 2014 at 9:27 AM, Mauro Bartolomeoli <mauro.bartolomeoli@anonymised.com> wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

+1, about time!

Cheers
Andrea

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Ing. Andrea Aime

@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


The printing module needs a bit more documentation before being an extension…

The current page (http://docs.geoserver.org/stable/en/user/community/printing/) just punts readers over to the GeoExt site. Actually I am mistaken, it links to a search of the GeoEXT site: http://geoext.org/search.html?q=print

I have run a couple training courses that showed example YAML files and similar, I think even one example would help?

···

Jody Garnett

On Thu, Mar 27, 2014 at 7:27 PM, Mauro Bartolomeoli <mauro.bartolomeoli@anonymised.com> wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it




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

+0 The proposal doesnt seem to be referencing the requirements for promoting a community module. http://docs.geoserver.org/stable/en/developer/policies/community-modules.html
  Is no. 5 and 7 satisfied for instance?

Notice: This email and any attachments are confidential.
If received in error please destroy and immediately notify us.
Do not copy or disclose the contents.

Hi Jody,
I think I can improve the documentation a bit and include links to the official MapFish-print documentation (http://www.mapfish.org/doc/print/index.html) that is, in my opinion, a good reference.
Do you think that could help?

Mauro

···

2014-03-27 12:04 GMT+01:00 Jody Garnett <jody.garnett@anonymised.com>:

The printing module needs a bit more documentation before being an extension…

The current page (http://docs.geoserver.org/stable/en/user/community/printing/) just punts readers over to the GeoExt site. Actually I am mistaken, it links to a search of the GeoEXT site: http://geoext.org/search.html?q=print

I have run a couple training courses that showed example YAML files and similar, I think even one example would help?

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


Jody Garnett

On Thu, Mar 27, 2014 at 7:27 PM, Mauro Bartolomeoli <mauro.bartolomeoli@anonymised.com> wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it




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

Hi Phil,
you are right, these are the requirements and my comments on them:

  1. The module has at least a “handful” of users

    Looking at our customers and mails on this mailing list I think there are already many users of the community module.

  2. The module has a designated and active maintainer

    I’m offering as the new mantainer for the module.

  3. The module is considered “stable” by the majority of the PSC

    PSC has to vote on this.

  4. The module maintains 40% test coverage

    This could be a weak point, I should check, but: the community module itself is really a very simple wrapper around MapFish print-lib that has its unit tests suite (I’ll try to get the test coverage of that, but since the build system is not using maven I have to check how to run cobertura on it).

  5. The module has no IP violations

    MapFish print-lib is GPL3.

  6. The module has a page in the user manual

    There is, and I am going to improve it in the process.

  7. The maintainer has signed the GeoServer Contributor Agreement

    If the PSC accepts me as a mantainer, I have signed the Agreement.

···

2014-03-27 21:36 GMT+01:00 Phil Scadden <p.scadden@anonymised.com>:

+0 The proposal doesnt seem to be referencing the requirements for
promoting a community module.
http://docs.geoserver.org/stable/en/developer/policies/community-modules.html
Is no. 5 and 7 satisfied for instance?

Notice: This email and any attachments are confidential.
If received in error please destroy and immediately notify us.
Do not copy or disclose the contents.



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

On Fri, Mar 28, 2014 at 9:01 AM, Mauro Bartolomeoli <
maurobartolomeoli@anonymised.com> wrote:

Hi Phil,
you are right, these are the requirements and my comments on them:

   1.

   *The module has at least a "handful" of users*

   Looking at our customers and mails on this mailing list I think there
   are already many users of the community module.
   2.

   *The module has a designated and active maintainer*

   I'm offering as the new mantainer for the module.
   3.

   *The module is considered "stable" by the majority of the PSC*

   PSC has to vote on this.
   4.

   *The module maintains 40% test coverage*

   This could be a weak point, I should check, but: the community module
   itself is really a very simple wrapper around MapFish print-lib that has
   its unit tests suite (I'll try to get the test coverage of that, but since
   the build system is not using maven I have to check how to run cobertura on
   it).
   5.

   *The module has no IP violations*

   MapFish print-lib is GPL3.
   6.

   *The module has a page in the user manual*

   There is, and I am going to improve it in the process.
   7.

   *The maintainer has signed the GeoServer Contributor Agreement*

   If the PSC accepts me as a mantainer, I have signed the Agreement.

Mauro, I suggest you add these to the proposal.

And yes, we need to do something about the tests, they just have to check
the integration works,
not that mapfish itself does

Cheers
Andrea

--

Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------

That would be good,

I would like if someone can follow our docs and generate a PDF with a title and/or watermark (i.e. the 90% use case). Linking to the map fish documentation for advanced functionality (or as a reference) would be fine.

Other than that it seems you are chewing through the extension requirements :slight_smile:

···

Jody Garnett

On Fri, Mar 28, 2014 at 6:55 PM, Mauro Bartolomeoli <mauro.bartolomeoli@anonymised.com> wrote:

Hi Jody,
I think I can improve the documentation a bit and include links to the official MapFish-print documentation (http://www.mapfish.org/doc/print/index.html) that is, in my opinion, a good reference.
Do you think that could help?

Mauro

2014-03-27 12:04 GMT+01:00 Jody Garnett <jody.garnett@anonymised.com>:

The printing module needs a bit more documentation before being an extension…

The current page (http://docs.geoserver.org/stable/en/user/community/printing/) just punts readers over to the GeoExt site. Actually I am mistaken, it links to a search of the GeoEXT site: http://geoext.org/search.html?q=print

I have run a couple training courses that showed example YAML files and similar, I think even one example would help?

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


Jody Garnett

On Thu, Mar 27, 2014 at 7:27 PM, Mauro Bartolomeoli <mauro.bartolomeoli@anonymised.com8…> wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it




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

Thanks Mauro, these are details I was looking for.

Notice: This email and any attachments are confidential.
If received in error please destroy and immediately notify us.
Do not copy or disclose the contents.

+0. I am pleased with your show of support; only my lack of familiarity with this module prevents me from giving it a +1, but I will not stand in your way!

GeoServer is GPL-v2-or-later which is compatible with GPLv3:
http://www.gnu.org/licenses/quick-guide-gplv3.html

Kind regards,
Ben.

On 27/03/14 16:27, Mauro Bartolomeoli wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110+-+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

--

Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------

------------------------------------------------------------------------------

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

--
Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com>
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

+1

···

On Mon, Mar 31, 2014 at 2:07 AM, Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com> wrote:

+0. I am pleased with your show of support; only my lack of familiarity
with this module prevents me from giving it a +1, but I will not stand
in your way!

GeoServer is GPL-v2-or-later which is compatible with GPLv3:
http://www.gnu.org/licenses/quick-guide-gplv3.html

Kind regards,
Ben.

On 27/03/14 16:27, Mauro Bartolomeoli wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it




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


Ben Caradoc-Davies Ben.Caradoc-Davies@anonymised.com
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre



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

Justin Deoliveira
Vice President, Engineering | Boundless
jdeolive@anonymised.com
@j_deolive

Hi everybody, I prepared the pull request for the promotion and mapfish print-lib upgrade to 2.0.0.

The pull request is here: https://github.com/geoserver/geoserver/pull/557

I also did some small improvements:

  • changed the demo page to use the standard topp:states from the running geoserver instead of an external layer
  • included print-lib 2.0.0 documentation as rst pages in the geoserver documentation
  • added unit tests to grow coverage to 78%

Someone wishing to review?

Thanks
Mauro

···

2014-03-31 16:03 GMT+02:00 Justin Deoliveira <jdeolive@anonymised.com>:

+1

==
Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it


On Mon, Mar 31, 2014 at 2:07 AM, Ben Caradoc-Davies <Ben.Caradoc-Davies@anonymised.com> wrote:

+0. I am pleased with your show of support; only my lack of familiarity
with this module prevents me from giving it a +1, but I will not stand
in your way!

GeoServer is GPL-v2-or-later which is compatible with GPLv3:
http://www.gnu.org/licenses/quick-guide-gplv3.html

Kind regards,
Ben.

On 27/03/14 16:27, Mauro Bartolomeoli wrote:

Please, vote here:
http://geoserver.org/display/GEOS/GSIP+110±+Promoting+the+Printing+community+module+to+extension

Thanks
Mauro

Meet us at GEO Business 2014! in London! Visit http://goo.gl/fES3aK
for more information.

Dott. Mauro Bartolomeoli
@mauro_bart
Senior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it




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


Ben Caradoc-Davies Ben.Caradoc-Davies@anonymised.com
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre



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

Justin Deoliveira
Vice President, Engineering | Boundless
jdeolive@anonymised.com
@j_deolive