Re: [Geotools-devel] Incoherent getNames() and getFeatureSource() behavior in AppSchemaDataAccess?

2015-08-07 Thread Niels Charlier
No idea. Seems either you, Rini or Gabriel is responsible ;) Niels; On 08/07/2015 06:31 AM, Ben Caradoc-Davies wrote: > Stefano, here is the change I tested: > https://github.com/geotools/geotools/compare/master...bencaradocdavies:getfeaturesource > > > > So it looks like there are some expecta

[Geotools-devel] DEMStore community module

2016-02-10 Thread Niels Charlier
I'd like permission to create a new community module called "demstore". This is a store specialised for Digital Elevation Models. The idea is to merge different granules that may be of different resolution and quality. It will be built on top of the mosaicstore with extra features such as - mor

[Geotools-devel] geotools unsupported module

2016-02-10 Thread Niels Charlier
I'd like permission to create a new unsupported module called "demstore". This is a store specialised for Digital Elevation Models. The idea is to merge different granules that may be of different resolution and quality. It will be built on top of the mosaicstore with extra features such as - m

Re: [Geotools-devel] DEMStore community module

2016-02-10 Thread Niels Charlier
sorry, one was sent to the wrong list, please ignore On 10-02-16 17:21, Niels Charlier wrote: > I'd like permission to create a new community module called "demstore". > > This is a store specialised for Digital Elevation Models. The idea is to > merge different granul

Re: [Geotools-devel] gt-coverage refactorings

2016-03-31 Thread Niels Charlier
I have looked at Daniele's PR (superficially) and it looks good, but I would expect a proposal is in order to move a lot of API from a plugin to a core module. As far as I can see, there is little or no overlap between the two bodies of work at the moment. I don't expect any issues there. Co

Re: [Geotools-devel] gt-coverage refactorings

2016-04-04 Thread Niels Charlier
Hi Simone, Daniele, List, Here is what we currently have for our proposal: https://github.com/geotools/geotools/wiki/Refactor-ImageMosaic-for-extensibility. This is still is discussion phase, we'd like to hear your input :) Kind Regards Niels On 31-03-16 20:14, Niels Charlier wro

Re: [Geotools-devel] ImageMosaic API refactor proposal

2016-04-06 Thread Niels Charlier
Hello Devon, Directory walking is part of the harvesting process, which is at the moment implemented inside the ImageMosaicReader (see the inner class HarvestedResource). In the API, there is a separate MosaicHarvester interface, as well as a very light ImageMosaicReader with all the logic ins

Re: [Geotools-devel] ImageMosaic API refactor proposal

2016-04-16 Thread Niels Charlier
fax or e-mail and delete the information in this message > that has been received in error. The sender does not give any warranty > or accept liability as the content, accuracy or completeness of sent > messages and accepts no responsibility for changes made after they > were sent or

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-04-20 Thread Niels Charlier
Issue number (2) was unfortunately introduced when adding the complex store last year. Both of these issues can probably be resolved without *too* much effort. Of course we can't know for sure, but I am 95% convinced that once those few annoying roadblocks to using wfs-ng are resolved, it will

Re: [Geotools-devel] ImageMosaic API refactor proposal

2016-04-21 Thread Niels Charlier
On 17-04-16 04:17, Jody Garnett wrote: > This proposal (and email thread) should be orthogonal to the RnD > discussion on setting up a mosaic with multiple projections. > > This mosaic is focused on creating/managing the index used. One part I > cannot wrap my head around is the workflow behind t

Re: [Geotools-devel] ImageMosaic API refactor proposal

2016-04-22 Thread Niels Charlier
anules) two different functions altogether. -- Jody Garnett On 21 April 2016 at 03:00, Niels Charlier <mailto:ni...@scitus.be>> wrote: On 17-04-16 04:17, Jody Garnett wrote: This proposal (and email thread) should be orthogonal to the RnD discussion on setting u

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-04-22 Thread Niels Charlier
Andre, The reason I closed it as "not a bug" was because I knew it was done by intentional design (not by me, but by the initial developer of wfs-ng) for a particular purpose and therefore I did not consider it a bug. You do make good points to criticise this choice, which does affect backwar

Re: [Geotools-devel] ImageMosaic API refactor proposal

2016-05-23 Thread Niels Charlier
b.com/dvntucker/geotools/wiki/Refactor-ImageMosaic-for-extensibility Cheers, Devon On Fri, Apr 22, 2016 at 1:20 AM, Niels Charlier <mailto:ni...@scitus.be>> wrote: On 21-04-16 20:01, Jody Garnett wrote: Thanks for the clarification Niels, I was not aware that Prop was

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-06-24 Thread Niels Charlier
I'm digging up the WFS/WFS-NG datastore fight and want to sort it out once and for all. I am compiling a comprehensive list of what needs to happen for WFS-NG to become acceptable: https://github.com/geotools/geotools/wiki/WFS-to-WFS-NG-upgrade Any feedback/additions to this list are more than w

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-06-24 Thread Niels Charlier
On 24-06-16 15:36, Andrea Aime wrote: I believe I sent you a list of mapserver and tinyows servers that were buggy vs the WFS spec and that the wfs store was configurable to work with anyways (as you may imagine, going to the server owner and asking for an upgrade or a different interpretation o

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-06-27 Thread Niels Charlier
On 24-06-16 17:10, Andrea Aime wrote: > Anyways, I've found again the links to the caps documents of the > mapserver and a tinyows in question... no idea if the problems are > still there, years have passed by and the servers might have been > upgraded in the meantime: > > MapServer WFS: > http

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-07-18 Thread Niels Charlier
Andrea, This MapServer WFS url doesn't work any more. Now I can't test what was going wrong there ... Do you know where it went or if I can somehow test it still? Regards Niels On 27-06-16 14:44, Niels Charlier wrote: > On 24-06-16 17:10, Andrea Aime wrote: >> Anyways, I

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-07-21 Thread Niels Charlier
shows the same misbehaviors as the Tuscany one did: http://demo.mapserver.org/cgi-bin/wfs?SERVICE=WFS&REQUEST=GetCapabilities Cheers Andrea On Mon, Jul 18, 2016 at 10:06 AM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Andrea, This MapServer WFS url doesn't work an

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-07-21 Thread Niels Charlier
Thanks Andrea, good idea On 21-07-16 14:15, Andrea Aime wrote: I don't know personally, but asked on mapserver-devel: http://osgeo-org.1560.x6.nabble.com/Support-for-wfs-2-0-on-mapserver-demo-td5277563.html Cheers Andrea On Thu, Jul 21, 2016 at 2:07 PM, Niels Charlier <mailto:ni...@s

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-07-25 Thread Niels Charlier
Hi Jody, I've added a patch for WFS2.0 in MapServer to the existing PR. A couple more fixes are coming. Regards Niels On 22-07-16 18:45, Jody Garnett wrote: Thanks for the positive feedback, Niels has a couple of pull requests ready and then we should be able to complete the "migration" ...

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-07-26 Thread Niels Charlier
anticipation of your success fixing wfs-ng. -- Jody Garnett On 25 July 2016 at 12:35, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hi Jody, I've added a patch for WFS2.0 in MapServer to the existing PR. A couple more fixes are coming. Regards Niels

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-07-29 Thread Niels Charlier
copy the test to core (since it is testing the functionality of the geoserver catalog) - it may also be fine as a community module (as you indicated) to verify the change of functionality; it can be removed after the transition is complete -- Jody Garnett On 26 July 2016 at 05:16, Niels Cha

Re: [Geotools-devel] wfs-ng vs wfs datastore fight!

2016-08-17 Thread Niels Charlier
://github.com/geotools/geotools/wiki/WFS-to-WFS-NG-upgrade) Kind Regards Niels On 29-07-16 16:00, Niels Charlier wrote: You're right, I can just put it in main,seeing it already depends on wfs-ng. See https://github.com/geoserver/geoserver/pull/1724 Regards Niels On 26-07-16 22:06,

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-09-08 Thread Niels Charlier
instead of Otherwise the one with the earliest time coordinates is last in the list instead of first... and geotools fails on this. Is this normal? This issue is definitely unrelated to netcdf-java, but I am not sure whether this is a bug in geotools or in the .ncml file. Regards Niels On

Re: [Geotools-devel] App-Schema: Encoding of Filters on Nested Attributes

2016-09-13 Thread Niels Charlier
en loaded. For those who might be unfamiliar with it, the inner workings of the current joining implementation are clearly illustrated by Niels Charlier in this presentation: https://www.seegrid.csiro.au/wiki/pub/Infosrvices/GeoserverAppSchemaJoining/joining_presentation.pdf The aforementioned prese

Re: [Geotools-devel] App-Schema: Encoding of Filters on Nested Attributes

2016-09-14 Thread Niels Charlier
On Tue, Sep 13, 2016 at 11:17 AM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hi Stefano, I think it is great you added that feature! A vast improvement to the work that I always had hoped would still be developed. Out of curiosity, why did you choose the "wh

[Geotools-devel] netcdf issue: VariableAdapter returns wrong time index

2016-09-20 Thread Niels Charlier
See https://osgeo-org.atlassian.net/browse/GEOT-5515 Consider the following line in VariableAdapter https://github.com/geotools/geotools/blob/master/modules/plugin/coverage-multidim/netcdf/src/main/java/org/geotools/imageio/netcdf/VariableAdapter.java#L1100 as compared to https://github.com/ge

[Geotools-devel] netcdf runtime dimension question

2016-09-23 Thread Niels Charlier
Hello Daniele, As you know, I am working on support for aggregation on runtime dimension in netcdf. I am now looking into what needs to happen in VariableAdaptor. I am very new at netcdf, so I do not yet understand everything that is happening to the fullest extent in this class. I have found

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-09-23 Thread Niels Charlier
order, so would you agree if I changed it to that regards? Niels On 08-09-16 14:51, Niels Charlier wrote: Ben, Sean, I am also including the geotools list in the discussion, because it regards both libraries. I got the aggregation file to work. Apart from some obvious bug fixes and additio

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-09-23 Thread Niels Charlier
dimension (which is the case here)? Perhaps the issue is somewhere else in the code, and the values should be sorted after reading the list of files. Kind Regards Niels On 23-09-16 15:50, Daniele Romagnoli wrote: Hi Niels, On Fri, Sep 23, 2016 at 2:40 PM, Niels Charlier <mailto:ni...@scitus

Re: [Geotools-devel] netcdf runtime dimension question

2016-09-23 Thread Niels Charlier
Index machinery or remove that in favor of a better management of additional dimensions. Does it helps? Cheers, Daniele On Fri, Sep 23, 2016 at 2:34 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello Daniele, As you know, I am working on support for aggregat

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-09-28 Thread Niels Charlier
On 23-09-16 17:03, Daniele Romagnoli wrote: I think that once the aggregation is in place, times can be in any order as you reported, so I think it's time to revisit the timeVariable min/max computation (not sure there are other methods around based on that supposition. I didn't check it yet).

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-09-28 Thread Niels Charlier
getMaximum() is called. But then again, I wonder if we shouldn't do the same for non-time values (regular CoordinateValue). What do you think? Regards Niels On 28-09-16 11:41, Daniele Romagnoli wrote: Hi Niels, On Wed, Sep 28, 2016 at 11:05 AM, Niels Charlier <mailto:ni...@scitus.be

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-09-30 Thread Niels Charlier
agree? What are your thoughts? Regards Niels On 28-09-16 13:44, Niels Charlier wrote: Yeah, still that implies that the order of the files specified in the aggregation must be the same as the order of values for the aggregated dimension inside those files themselves. I can find no such spec

Re: [Geotools-devel] Support two dimensional NetCDF coordinate variables

2016-10-05 Thread Niels Charlier
Hello Iain, Did you ever carry out your plans to support two dimension coordinate variables? Kind Regards Niels On 06/20/2016 04:55 AM, Iain Matcham wrote: > Hi > > I have a number of netcdf files that I am trying to read with > gt-netcdf. However they use two dimensional coordinate variable

Re: [Geotools-devel] [netcdf-java] runtime aggregation

2016-10-07 Thread Niels Charlier
support pull request: https://github.com/geotools/geotools/pull/1296/commits/88bc830e570d805a3cd338e2d8dc2abde8d4fab7 https://github.com/geotools/geotools/pull/1296 Regards Niels On 30-09-16 12:24, Niels Charlier wrote: Hi Daniele, Did you have any more comments on my last email? Would it make

Re: [Geotools-devel] Support two dimensional NetCDF coordinate variables

2016-10-12 Thread Niels Charlier
than happy push my management to > be allowed to continue my work on this. > > Thanks > Iain > > On 5/10/2016 10:18 p.m., Niels Charlier wrote: >> Hello Iain, >> >> Did you ever carry out your plans to support two dimension coordinate >> variables? >>

Re: [Geotools-devel] Support two dimensional NetCDF coordinate variables

2016-10-14 Thread Niels Charlier
ot a netcdf expert ( my first main investigation of > the format was this exercise), so you may be able to improve on this! > > Kind regards > > Iain > > > On 12/10/2016 11:13 p.m., Niels Charlier wrote: >> Hello Iain, >> >> Thank you for your response. &g

[Geotools-devel] GeometryReducer

2016-11-09 Thread Niels Charlier
Hi Andrea, I have a question with regards to the GridGeometryReducer class, which you are credited with. Consider the following lines: https://github.com/geotools/geotools/blob/master/modules/library/render/src/main/java/org/geotools/renderer/lite/gridcoverage2d/GridGeometryReducer.java#L86 It

Re: [Geotools-devel] GeometryReducer

2016-11-18 Thread Niels Charlier
Andrea On Wed, Nov 9, 2016 at 2:46 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hi Andrea, I have a question with regards to the GridGeometryReducer class, which you are credited with. Consider the following lines: https://github.com/geotools/geotools/blo

[Geotools-devel] strange distortion from warp operation in ImageWorker

2016-11-21 Thread Niels Charlier
Hello, We are getting strange distortions when zooming on a very large mosaic (see https://osgeo-org.atlassian.net/browse/GEOS-7793), which appear to be caused by the JAI warp operation called in ImageWorker (see https://github.com/geotools/geotools/blob/master/modules/library/coverage/src/main

Re: [Geotools-devel] strange distortion from warp operation in ImageWorker

2016-11-21 Thread Niels Charlier
Hello Andrea, Thank you for your response. On 21-11-16 18:07, Andrea Aime wrote: Yes, a significant one under load testing, as it avoids the generation of one level of tiles in the chain, thus reducing both computation and load on the tile cache. So by turning it off you get the good looking

Re: [Geotools-devel] strange distortion from warp operation in ImageWorker

2016-11-22 Thread Niels Charlier
Indeed, makes sense that those are the two possible approaches to fix the problem. Regards Niels On 22-11-16 11:43, Andrea Aime wrote: On Mon, Nov 21, 2016 at 5:52 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Do you have an idea what could cause this kind of distortion?

Re: [Geotools-devel] [Geoserver-devel] jai-ext initialisation

2016-12-12 Thread Niels Charlier
Hello, I think this is a misunderstanding. I did not intend my email as a complaint or criticism (and especially not to make myself "feel better", I don't see it that way at all). It was merely intended as an entirely neutral issue report, like "hey, I found this thing that seemed to have sli

Re: [Geotools-devel] [Geoserver-devel] jai-ext initialisation

2016-12-16 Thread Niels Charlier
2-16 15:55, Andrea Aime wrote: On Mon, Dec 12, 2016 at 1:58 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: I have another question though - why is geoserver doing this thing in GeoServerInitStartupListener? I'm not just asking about the place it is doing, why is it doi

[Geotools-devel] netcdf runtime pull requests

2017-03-01 Thread Niels Charlier
Hello Simone, Daniele, I have made two pull requests with the aim of supporting "runtime" as an additional dimension in netcdf-java: The first is in the netcdf-java module itself: https://github.com/geotools/geotools/pull/1472 Rather than just adding runtime to the supported time and elevation

Re: [Geotools-devel] netcdf runtime pull requests

2017-03-01 Thread Niels Charlier
e 2.11-RC1 release, new features will have to wait for the master branch to switch to 18-snapshot/2.12-snapshot to be merged Cheers Andrea On Wed, Mar 1, 2017 at 11:43 AM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello Simone, Daniele, I have made two pull reque

[Geotools-devel] proposed change: coverage readers custom dimension data type conversion

2017-03-22 Thread Niels Charlier
Hello, This message is particularly aimed at people who develop/use coverage readers that support custom dimensions, but are outside of the main geotools/geoserver code base. I am proposing a change that may affect them (see https://osgeo-org.atlassian.net/browse/GEOS-7989). In short: At this

Re: [Geotools-devel] [Geoserver-devel] proposed change: coverage readers custom dimension data type conversion

2017-03-28 Thread Niels Charlier
him along with another fellow at GDIT, they might be affected (or not). I also know GeoMesa has their own custom readers, no idea if they are dimension enabled or not. Cc'ed Jim just in case. Cheers Andrea On Wed, Mar 22, 2017 at 2:27 PM, Niels Charlier <mailto:ni...@scitus.

[Geotools-devel] filter function "not"

2019-04-30 Thread Niels Charlier
Hello, I discovered that the implemented and documented filter function "not" doesn't actually work. The parser treats the word "not" as a reserved word, used for filters, therefore it does not manage to parse it as an identifier inside an expression. Perhaps I am missing something, but ot

Re: [Geotools-devel] filter function "not"

2019-04-30 Thread Niels Charlier
e. On 30/04/2019 17:31, Andrea Aime wrote: On Tue, Apr 30, 2019 at 11:25 AM Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello, I discovered that the implemented and documented filter function "not" doesn't actually work. In what context? See belo

Re: [Geotools-devel] filter function "not"

2019-05-01 Thread Niels Charlier
otools.org/ECQL-Parser-Design_110493908.html We already have some special cases to recognize odd patterns (and encode odd patterns) for ECQL constructs like "property IN (1,2,3)" which is mapped to filter "in(1,2,3) == TRUE" -- Jody Garnett On Tue, 30 Apr 2019 at 14

Re: [Geotools-devel] unsupported kml datastore, no profile available

2020-02-09 Thread Niels Charlier
I don't mind personally but Ian's recent fixes/improvements seems to suggest he or someone is using it. Kind Regards Niels On 09/02/2020 13:02, Jody Garnett wrote: I have been arguing with IntiliJ and the geotools build for a bit this afternoon, one thing I noticed was your KML DataStore

Re: [Geotools-devel] unsupported kml datastore, no profile available

2020-02-09 Thread Niels Charlier
. Kind Regards Niels On 09/02/2020 14:24, Jody Garnett wrote: Your name is on the work, so I am asking you :) On Sun, Feb 9, 2020 at 2:20 PM Niels Charlier <mailto:ni...@scitus.be>> wrote: I don't mind personally but Ian's recent fixes/improvements seems to suggest he or

[Geotools-devel] ogc filter 1.1 doesn't support matchCase in PropertyIsLike

2013-09-24 Thread Niels Charlier
Hello, While testing the geoserver csw module, Bart van den Eijnden discovered that the ogc 1.1 filter parser does not support matchCase in a PropertyIsLike filter. For example this filter: ogc:Filter xmlns:ogc="http://www.opengis.net/ogc";>matchCase="false" wildCard="*" singleChar="." escap

Re: [Geotools-devel] ogc filter 1.1 doesn't support matchCase in PropertyIsLike

2013-09-24 Thread Niels Charlier
On 24/09/13 15:04, Andrea Aime wrote: On Tue, Sep 24, 2013 at 2:21 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello, While testing the geoserver csw module, Bart van den Eijnden discovered that the ogc 1.1 filter parser does not support matchCase in a Prope

Re: [Geotools-devel] ogc filter 1.1 doesn't support matchCase in PropertyIsLike

2013-09-24 Thread Niels Charlier
On 24/09/13 15:34, Niels Charlier wrote: Apparently the OGC did change its schema a while back to support this, it was initially not supported IIRC, but now it is. Although there is filter 2.0 now, should we not support this? I don't see a reason not to, besides maybe la

[Geotools-devel] kml datastore

2013-11-19 Thread Niels Charlier
Hi, I am proposing a new unsupported module that allowes kml files to be used as datastores in geoserver: http://docs.codehaus.org/display/GEOTOOLS/KML+DataStore https://github.com/geotools/geotools/pull/316 Regards, Niels -

Re: [Geotools-devel] kml datastore

2013-11-20 Thread Niels Charlier
On 19/11/13 15:27, Andrea Aime wrote: On Tue, Nov 19, 2013 at 2:56 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hi, I am proposing a new unsupported module that allowes kml files to be used as datastores in geoserver: http://docs.codehaus.org/display/GEO

[Geotools-devel] mbtiles community module

2013-12-03 Thread Niels Charlier
Hi everyone, I would like to add a community module for the mbtiles format, similar to the geopkg module. https://github.com/NielsCharlier/geotools/commit/fdfcb3a98d84711a40b79ff1924ae26aa605485f Regards Niels -- Spons

[Geotools-devel] mbtiles mosaic

2013-12-04 Thread Niels Charlier
Hello list, I am starting to work on build a mosaic coverage reader for an mbtiles database. I am suspecting that the 'mosaicing' logic is already present in geotools. I was wondering if anyone familiar with image-mosaic and image-mosaic-jdbc modules can give me any tips on what to use / point

[Geotools-devel] multiple coverages in a grid coverage reader

2014-01-14 Thread Niels Charlier
Hello Mailing List, I am writing a geopkg coverage reader. A geopkg can have many coverages, each of which can have their own envelope, CRS, resolutions, ... Currently the API works as follows. - The GridFormatFactorySpi Service returns a AbstractGridFormat (not the GridFormat interface but an

[Geotools-devel] Encode z-dimension in GML2

2014-03-19 Thread Niels Charlier
Hello everyone, I propose the following change: https://github.com/geotools/geotools/pull/402 It allows the z-dimension to be encoded in GML2 whenever it is present in the data (it does now only if the CRS is explicitly 3D). This is already how it works for most of the other output formats suc

Re: [Geotools-devel] Encode z-dimension in GML2

2014-03-21 Thread Niels Charlier
On 20/03/14 09:38, Andrea Aime wrote: On Wed, Mar 19, 2014 at 11:17 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello everyone, I propose the following change: https://github.com/geotools/geotools/pull/402 It allows the z-dimension to be encoded in GML2 whene

[Geotools-devel] wfs-ng and AbstractDataStoreTest

2014-04-25 Thread Niels Charlier
Hello, While working on wfs-ng, I found the class AbstractDataStoreTest and I was wondering what the deal is with that class: - there is a copy inside the wfs-ng module, and one inside the main module. The first appears to be a modified version of the latter. Shouldn't the modified version move

[Geotools-devel] wfs-ng improvements

2014-05-07 Thread Niels Charlier
Hello, My task is to make wfs-ng a supported module; and also to enable transactions on a wfs-ng datastore that uses wfs 1.0/1.1. (Almost) everything I did is in the following branch: https://github.com/geotools/geotools/commits/wfs-ng-improvements This consists of the following changes: 1.

[Geotools-devel] gt-data events bug?

2014-05-07 Thread Niels Charlier
Hi Everyone, while working on wfs-ng, I think I found a bug in the events system in gt-data. This is my suggested change: https://github.com/NielsCharlier/geotools/commit/af9df3e282d0ad70fbf463fa5e2b36225a53116a In modules/library/data/src/main/java/org/geotools/data/store/ContentEntry.java

Re: [Geotools-devel] wfs-ng improvements

2014-05-07 Thread Niels Charlier
On 07/05/14 11:28, Andrea Aime wrote: > > I have a question, did you test the store interactively (e.g., from > udig/geoserver) against any WFS server, > if so, which ones? (as in type and version, don't need the caps urls) > So far I have only tested connecting to another geoserver. Kind Regard

Re: [Geotools-devel] gt-data events bug?

2014-05-07 Thread Niels Charlier
et this commit notification sent to its listeners The other case is transaction auto commit: a) event sent on transaction auto commit, such as feature add b) a second transaction, which should get this feature add event notification Jody Jody Garnett On Wed, May 7, 2014 at 7:15 AM, Niels Charlier &

Re: [Geotools-devel] gt-data events bug?

2014-05-08 Thread Niels Charlier
s On 08/05/14 05:31, Jody Garnett wrote: Your original bug report made sense, I was just going through the senario to use to test if fix works. Jody Garnett On Wed, May 7, 2014 at 10:52 AM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Jody, If the events need to

Re: [Geotools-devel] wfs-ng improvements

2014-05-19 Thread Niels Charlier
On 17/05/14 01:46, Jody Garnett wrote: > When adding gt-wfs-ng (and removing gt-wfs) I get the following > dependencies: > > [INFO] +- org.geotools:gt-wfs-ng:jar:12-SNAPSHOT:compile > [INFO] | \- xpp3:xpp3:jar:1.1.3.4.O:compile > > Is there anything else besides xpp3 that I need? This is all non

Re: [Geotools-devel] wfs-ng improvements

2014-05-20 Thread Niels Charlier
Actually xmlunit is also necessary for the tests. Kind Regards Niels On 19/05/14 10:22, Niels Charlier wrote: > On 17/05/14 01:46, Jody Garnett wrote: >> When adding gt-wfs-ng (and removing gt-wfs) I get the following >> dependencies: >> >> [INFO] +- or

Re: [Geotools-devel] wfs-ng improvements

2014-05-27 Thread Niels Charlier
Hi Jody, An answer to all your questions ( I think ). On 22/05/14 02:39, Jody Garnett wrote: 1) version negotiation how? Okay, so I have been looking at this GML3 issue, and I found where it happens is in the method getDefaultOutputFormat in AbstractWFSStrategy. What this method does is l

Re: [Geotools-devel] wfs-ng improvements

2014-05-28 Thread Niels Charlier
HI Jody, Just a remark first: I did not write any of the code we are writing about, I am only reporting what I find. I don't feel one way or the other about it, so if you reckon it needs to be changed it is fine for me, as long as you specify exactly what you want to change about it. I'm just

Re: [Geotools-devel] wfs-ng improvements

2014-05-30 Thread Niels Charlier
Hello Jody, Regarding the info: the getInfo that passes on this information is implemented in ContentDataStore in gt-data, a class written by you two (Jody and Justin). It seems to me you want me to pass on information from the GeoServer catalog, but how is that information even available fr

Re: [Geotools-devel] wfs-ng improvements

2014-05-30 Thread Niels Charlier
On 29/05/14 01:44, Jody Garnett wrote: > 1) access to version negotiation I'm with you now on this one. The only thing I want confirmation for: the method createGetCapabilitiesRequest doesn't appear to be called from anywhere inside the module. Is the idea that udig calls this method explicitly

Re: [Geotools-devel] wfs-ng improvements

2014-05-31 Thread Niels Charlier
n Fri, May 30, 2014 at 6:28 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello Jody, Regarding the info: the getInfo that passes on this information is implemented in ContentDataStore in gt-data, a class written by you two (Jody and Justin). It seems to me you w

Re: [Geotools-devel] wfs-ng improvements

2014-06-02 Thread Niels Charlier
On 30/05/14 14:06, Jody Garnett wrote: It would be extra cool to avoid duplication of logic, if you can provide another way to access the same information that is fine. I tried letting the WFSDataStore connect and then asking for the WFS Client - but I could never find a method to give me acc

Re: [Geotools-devel] wfs-ng improvements

2014-06-03 Thread Niels Charlier
Hi Jody, Justin, I updated the branch / pull request to fix the getInfo stuff. I tested from udig (works) as well as added unit tests for this functionality. As far as I know atm, everything is done as requested. I suggest we move the discussion to the github pull request, and start from ther

Re: [Geotools-devel] wfs-ng improvements

2014-06-03 Thread Niels Charlier
On 03/06/14 14:23, Andrea Aime wrote: On Tue, Jun 3, 2014 at 2:19 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: As far as I know atm, everything is done as requested. I suggest we move the discussion to the github pull request, and start from there. The goal is sti

Re: [Geotools-devel] wfs-ng improvements

2014-06-03 Thread Niels Charlier
On 03/06/14 14:36, Andrea Aime wrote: > > This might become a problem for GeoServer users that have WFS data > stores already configured in > GeoServer. Would it be difficult to create, say, a factory that acts > as a bridge for those? > > Also the old WFS store had a number of configurations abo

Re: [Geotools-devel] wfs-ng improvements

2014-06-03 Thread Niels Charlier
No, these appear not to be present in the wfs-ng module. Kind Regards Niels On 03/06/14 15:12, Andrea Aime wrote: On Tue, Jun 3, 2014 at 3:09 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Well I ported the tinyows and mapserver tests from wfs to wfs-ng... T

[Geotools-devel] wfs-ng parameter names

2014-06-05 Thread Niels Charlier
Hi Jody, You asked me a while ago to change the parameter names, because you wanted them to look nicer for humans in udig. But the key and the displayed name are the same. So for example, I changed "WFSDataStoreFactory:MAXFEATURES" to "Maximum Features" because of your request. But the keys ar

Re: [Geotools-devel] wfs-ng parameter names

2014-06-05 Thread Niels Charlier
I am also asking this question because I am adding those parameters Andrea was talking about and I was wondering if I should also change their name or not. If the idea is that wfs-ng replaces wfs with the same parameter configuration... Regards Niels On 05/06/14 11:34, Niels Charlier wrote

[Geotools-devel] Fwd: Re: wfs-ng parameter names

2014-06-05 Thread Niels Charlier
iption), false, 0, 1, 0, kvp); } I am on IRC if you need a hand ... Jody Garnett On Thu, Jun 5, 2014 at 7:34 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hi Jody, You asked me a while ago to change the parameter names, because you wanted them to look nicer

[Geotools-devel] wfs-ng support for 2.0

2014-06-10 Thread Niels Charlier
Hi Justin, I made an additional branch: https://github.com/geotools/geotools/compare/wfs-ng-improvements-2.0 which includes an additional patch to support wfs 2.0 in wfs-ng. I started working on that when that was still on my list as next thing to do. As I explained earlier, this does most to s

Re: [Geotools-devel] wfs-ng improvements

2014-06-10 Thread Niels Charlier
On 03/06/14 15:17, Andrea Aime wrote: On Tue, Jun 3, 2014 at 3:12 PM, Andrea Aime mailto:andrea.a...@geo-solutions.it>> wrote: The parameters I'm concerned, because they were recently (roughly one year ago) added, are the following: final String namespaceOverride = (String

Re: [Geotools-devel] wfs-ng improvements

2014-06-13 Thread Niels Charlier
we can squeeze out a few more hours we can port them over. On Tue, Jun 10, 2014 at 5:56 AM, Andrea Aime mailto:andrea.a...@geo-solutions.it>> wrote: On Tue, Jun 10, 2014 at 1:47 PM, Niels Charlier mailto:ni...@scitus.be>> wrote: On 03/06/14 15:17, Andrea Aime wrote:

Re: [Geotools-devel] gt-data events bug?

2014-06-30 Thread Niels Charlier
7:17 AM, Jody Garnett <mailto:jody.garn...@gmail.com>> wrote: Unfortantly we always need a test case when accepting fixes. Still open up a bug report and it can wait until one of us has time. Jody Garnett On Thu, May 8, 2014 at 6:32 AM, Niels Charlier mailto:ni...@

Re: [Geotools-devel] WFS 2.0 support for wfs_ng

2014-10-06 Thread Niels Charlier
Hello Michael, I am currently not working on the wfs-ng-2.0 branch any more because funding dried up for this project and priorities changed. I did however made quite some progress on wfs-ng-2.0 but there were still a few remaining issues. Unfortunately that work is just sitting there now. I'

Re: [Geotools-devel] WFS 2.0 support for wfs_ng

2014-10-10 Thread Niels Charlier
On 10/10/14 19:39, Jody Garnett wrote: Any idea why this library is considered such a lower priority? It’s not even a supported module. It doesn’t make sense to me. I think it is just a matter of funds/volunteers. WFS is not alone - I would love to see a GML DataStore produced, or

Re: [Geotools-devel] WFS 2.0 support for wfs_ng

2014-10-10 Thread Niels Charlier
n’t make sense to me. All, Does the community want/need a Java-based WFS client that works with 1.0.0, 1.1.0 and 2.0.0? Are there other Java-based clients? Thanks, Mike *From:*Niels Charlier [mailto:ni...@scitus.be] *Sent:* Monday, October 06, 2014 1:01 PM *To:* geotools-devel@lists.sourcefor

Re: [Geotools-devel] Severe slowdown in ContentDataStore in 12.x and trunk

2014-10-11 Thread Niels Charlier
Hi Andrea, Blimey... I would never have expected that little change to have such an effect :O :O That is extremely unfortunate. My apologies for this terrible mistake. However, I did not just commit this without a review. The commit was part of a bigger pull request: https://github.com/geot

[Geotools-devel] imagemosaic blank response issues

2014-11-25 Thread Niels Charlier
Hello, A client needed to configure an imagemosaic datastore with a custom made mosaic configuration .properties file. They were having issues: with native JAI the tiled image was showing black boxes and if transparentcolor was set to #000 an "java.lang.IllegalStateException: Unsupported da

Re: [Geotools-devel] GeoPackage and QGIS/GDAL Compatibility

2014-12-29 Thread Niels Charlier
On 28-12-14 19:10, Justin Deoliveira wrote: As for updating bounds in gpkg_contents automatically I am thinking that the driver should put some triggers in place to do this. We could re-use the functions that Niels implemented for spatial indexing capabilities to pull out the min/max x/y fro

[Geotools-devel] wfs 2.0 in wfs-ng module

2015-01-13 Thread Niels Charlier
Hello List, I finished something I started last year but never finished: enable WFS 2.0 in the wfs-ng module. Some of the work I had already prepared was sitting in a separate branch called wfs-ng-improvements-2.0. Some of the work done in this branch had in the meantime been done a second time

Re: [Geotools-devel] wfs 2.0 in wfs-ng module

2015-01-14 Thread Niels Charlier
oposing this feature for the next stable release? Sampo On Tue, Jan 13, 2015 at 5:27 PM, Niels Charlier <mailto:ni...@scitus.be>> wrote: Hello List, I finished something I started last year but never finished: enable WFS 2.0 in the wfs-ng module. Some of the wo

Re: [Geotools-devel] wfs 2.0 in wfs-ng module

2015-01-15 Thread Niels Charlier
Capabilities file. If it works for you, then I guess something goes > wrong when rebasing the branch. > > > Sampo > > > > > On Thu, Jan 15, 2015 at 12:42 PM, Niels Charlier wrote: > >> Hello Sampo, >> >> Can you clarify how I can reproduce this

Re: [Geotools-devel] wfs 2.0 in wfs-ng module

2015-01-15 Thread Niels Charlier
I have done so. Regards Niels On 15-01-15 18:21, Sampo Savolainen wrote: > Hi, > > Can you push the rebased branch so I can test the exact same version? > > Sampo > On Jan 15, 2015 5:37 PM, "Niels Charlier" wrote: > >> Hello Sampo, >> >> I canno

Re: [Geotools-devel] wfs 2.0 in wfs-ng module

2015-01-16 Thread Niels Charlier
the schema. Is there > any chance you have time to check this? > > > Sampo > > On Thu, Jan 15, 2015 at 9:03 PM, Niels Charlier wrote: > >> I have done so. >> >> Regards >> Niels >> >> >> On 15-01-15 18:21, Sampo Savolainen wrote: >

  1   2   3   >