[Geoserver-devel] Modification on the GeoPackage WPS process

GeoServer GeoServer Developer
1

Hi all,

I am working with the WPS GeoPackageProcess in the gs-geopkg community module and I was asking if I can do some modifications/fix to the code:

  • configure the path for the directory of the .geopkg file to create by adding a new optional field in the input GeoPackageProcessRequest object or a new WPS parameter.
  • make configurable the removal of the .geopkg files created. Also for this I would create another optional field in the GeoPackageProcessRequest object or another WPS parameter.
  • call the Geopackage.init() method after creating the GeoPackage object wrapping the input file(line 104). I would add this method because I have found an exception when trying to create the new .geopkg file.
  • define the input GetMap request CRS from the SRS specified by the GeoPackageProcessRequest object, because I have found that the CRS it is not set by default and an exception can be thrown if the BoundingBox CRS is not WGS84.
    If you have any question, I would be pleased to answer.

Regards,
Nicola Lagomarsini.

–

==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.

Ing. Nicola Lagomarsini
Junior 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

2

Hey Nicola,

These all seems like great improvements. +1 !

-Justin

···

On Fri, Sep 5, 2014 at 8:43 AM, Nicola Lagomarsini <nicola.lagomarsini@anonymised.com> wrote:

Hi all,

I am working with the WPS GeoPackageProcess in the gs-geopkg community module and I was asking if I can do some modifications/fix to the code:

  • configure the path for the directory of the .geopkg file to create by adding a new optional field in the input GeoPackageProcessRequest object or a new WPS parameter.
  • make configurable the removal of the .geopkg files created. Also for this I would create another optional field in the GeoPackageProcessRequest object or another WPS parameter.
  • call the Geopackage.init() method after creating the GeoPackage object wrapping the input file(line 104). I would add this method because I have found an exception when trying to create the new .geopkg file.
  • define the input GetMap request CRS from the SRS specified by the GeoPackageProcessRequest object, because I have found that the CRS it is not set by default and an exception can be thrown if the BoundingBox CRS is not WGS84.
    If you have any question, I would be pleased to answer.

Regards,
Nicola Lagomarsini.

–

==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.

Ing. Nicola Lagomarsini
Junior 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

Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/

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

–

Justin Deoliveira
VP Engineering | Boundless
jdeolive@anonymised.com
@boundlessgeo

3

Hi Justin,

when you have time, could you please take a look at the following pull requests.

These are related to GeoPackage. They introduce the support for an external configurable path and the possibility to configure a file as temporary.

This one contains a minor fix for the MBTilesGetMapOutputFormat when trying to reproject the request bbox and modifies the MBTilesFileWrapper in order to accept an external file.

Thanks.

Regards,
Nicola Lagomarsini.

···

2014-09-05 16:48 GMT+02:00 Justin Deoliveira <jdeolive@anonymised.com3839…>:

Hey Nicola,

These all seems like great improvements. +1 !

-Justin

–

==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.

Ing. Nicola Lagomarsini
Junior 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 Fri, Sep 5, 2014 at 8:43 AM, Nicola Lagomarsini <nicola.lagomarsini@anonymised.com> wrote:

Hi all,

I am working with the WPS GeoPackageProcess in the gs-geopkg community module and I was asking if I can do some modifications/fix to the code:

  • configure the path for the directory of the .geopkg file to create by adding a new optional field in the input GeoPackageProcessRequest object or a new WPS parameter.
  • make configurable the removal of the .geopkg files created. Also for this I would create another optional field in the GeoPackageProcessRequest object or another WPS parameter.
  • call the Geopackage.init() method after creating the GeoPackage object wrapping the input file(line 104). I would add this method because I have found an exception when trying to create the new .geopkg file.
  • define the input GetMap request CRS from the SRS specified by the GeoPackageProcessRequest object, because I have found that the CRS it is not set by default and an exception can be thrown if the BoundingBox CRS is not WGS84.
    If you have any question, I would be pleased to answer.

Regards,
Nicola Lagomarsini.

–

==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.

Ing. Nicola Lagomarsini
Junior 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

Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/

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

–

Justin Deoliveira
VP Engineering | Boundless
jdeolive@anonymised.com
@boundlessgeo

4

Looked over the pull request, all look good Nicola. Merge at your discretion.

···

On Wed, Sep 10, 2014 at 8:39 AM, Nicola Lagomarsini <nicola.lagomarsini@anonymised.com> wrote:

Hi Justin,

when you have time, could you please take a look at the following pull requests.

These are related to GeoPackage. They introduce the support for an external configurable path and the possibility to configure a file as temporary.

This one contains a minor fix for the MBTilesGetMapOutputFormat when trying to reproject the request bbox and modifies the MBTilesFileWrapper in order to accept an external file.

Thanks.

Regards,
Nicola Lagomarsini.

–

Justin Deoliveira
VP Engineering | Boundless
jdeolive@anonymised.com
@boundlessgeo

2014-09-05 16:48 GMT+02:00 Justin Deoliveira <jdeolive@anonymised.com>:

Hey Nicola,

These all seems like great improvements. +1 !

-Justin

–

==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.

Ing. Nicola Lagomarsini
Junior 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 Fri, Sep 5, 2014 at 8:43 AM, Nicola Lagomarsini <nicola.lagomarsini@anonymised.com> wrote:

Hi all,

I am working with the WPS GeoPackageProcess in the gs-geopkg community module and I was asking if I can do some modifications/fix to the code:

  • configure the path for the directory of the .geopkg file to create by adding a new optional field in the input GeoPackageProcessRequest object or a new WPS parameter.
  • make configurable the removal of the .geopkg files created. Also for this I would create another optional field in the GeoPackageProcessRequest object or another WPS parameter.
  • call the Geopackage.init() method after creating the GeoPackage object wrapping the input file(line 104). I would add this method because I have found an exception when trying to create the new .geopkg file.
  • define the input GetMap request CRS from the SRS specified by the GeoPackageProcessRequest object, because I have found that the CRS it is not set by default and an exception can be thrown if the BoundingBox CRS is not WGS84.
    If you have any question, I would be pleased to answer.

Regards,
Nicola Lagomarsini.

–

==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 for more information.

Ing. Nicola Lagomarsini
Junior 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

Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/

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

–

Justin Deoliveira
VP Engineering | Boundless
jdeolive@anonymised.com
@boundlessgeo