Re: [Geotools-devel] App-schema, xlinks, and curves
It's also done in gml2/GMLEncodingUtils.GeometryPropertyType_getProperty. Cheers Rini -Original Message- From: Caradoc-Davies, Ben (Mineral Resources, Kensington) Sent: Tuesday, 15 July 2014 12:25 PM To: Andrea Aime Cc: Geotools-Devel list; Angreani, Rini (Mineral Resources, Kensington) Subject: Re: [Geotools-devel] App-schema, xlinks, and curves Andrea, I am investigating now. My understanding is that the correct behaviour is to encode no content as this property is being mapped by reference. The handling of automatic (duplicate id) href is in ComplexSupportXSAnyTypeBinding and its neighbours, but there may be assumptions based on the behaviour of other bindings. Content is suppressed by setting the children to an empty container. I will have to check to see where mapped href is set differently and what changes there are to the geometry bindings. Kind regards, Ben. On 11/07/14 18:51, Andrea Aime wrote: Hi all, the arcs pull request has been updated to handle GML encoding of arcs (see the other mail about this topic for details). Generally speaking it's working well, GeoServer builds also work fine minus one hitch in the app-schema tests Basically, I have a xlink test failing on me, XLinkGeometryTest.testGeometry, that ends up encoding something like this: ex:multisurface xlink:href=#geom10/ ex:multipolygon xlink:href=#geom11/ ex:geometryref xlink:href=xlinkvalue1/ ex:curveref xlink:href=xlinkvalue2 gml:LineString gml:posList/ /gml:LineString /ex:curveref ... In other words, the other elements are empty, but curve is not. However, I don't see where the href management is, and how it's supposed to behave. Looking at the results, it seems the encoder was fed with an empty linestring... Do you have any quick pointers? Cheers Andrea -- == GeoServer Professional Services from the experts! Visit http://goo.gl/NWWaa2 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 --- -- Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
Re: [Geotools-devel] App-schema, xlinks, and curves
Thanks, Rini. I think I found the problem: there is an unwanted @Override in org.geotools.gml3.bindings.ext.CurvePropertyTypeBinding that prematurely returns the geometry object before the superclass has a chance to recognise that it should be nulled out because there is an xlink:href. I will test with a full build. Kind regards, Ben. On 15/07/14 14:11, Angreani, Rini (Mineral Resources, Kensington) wrote: It's also done in gml2/GMLEncodingUtils.GeometryPropertyType_getProperty. Cheers Rini -Original Message- From: Caradoc-Davies, Ben (Mineral Resources, Kensington) Sent: Tuesday, 15 July 2014 12:25 PM To: Andrea Aime Cc: Geotools-Devel list; Angreani, Rini (Mineral Resources, Kensington) Subject: Re: [Geotools-devel] App-schema, xlinks, and curves Andrea, I am investigating now. My understanding is that the correct behaviour is to encode no content as this property is being mapped by reference. The handling of automatic (duplicate id) href is in ComplexSupportXSAnyTypeBinding and its neighbours, but there may be assumptions based on the behaviour of other bindings. Content is suppressed by setting the children to an empty container. I will have to check to see where mapped href is set differently and what changes there are to the geometry bindings. Kind regards, Ben. On 11/07/14 18:51, Andrea Aime wrote: Hi all, the arcs pull request has been updated to handle GML encoding of arcs (see the other mail about this topic for details). Generally speaking it's working well, GeoServer builds also work fine minus one hitch in the app-schema tests Basically, I have a xlink test failing on me, XLinkGeometryTest.testGeometry, that ends up encoding something like this: ex:multisurface xlink:href=#geom10/ ex:multipolygon xlink:href=#geom11/ ex:geometryref xlink:href=xlinkvalue1/ ex:curveref xlink:href=xlinkvalue2 gml:LineString gml:posList/ /gml:LineString /ex:curveref ... In other words, the other elements are empty, but curve is not. However, I don't see where the href management is, and how it's supposed to behave. Looking at the results, it seems the encoder was fed with an empty linestring... Do you have any quick pointers? Cheers Andrea -- == GeoServer Professional Services from the experts! Visit http://goo.gl/NWWaa2 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 --- -- Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
Re: [Geotools-devel] App-schema, xlinks, and curves
And I am not sure that the getProperty @Override in org.geotools.gml3.bindings.ext.SurfacePropertyTypeBinding still makes sense. But fixing this is not required to fix the GeoServer build failure with arc support. Kind regards, Ben. On 15/07/14 14:19, Ben Caradoc-Davies wrote: Thanks, Rini. I think I found the problem: there is an unwanted @Override in org.geotools.gml3.bindings.ext.CurvePropertyTypeBinding that prematurely returns the geometry object before the superclass has a chance to recognise that it should be nulled out because there is an xlink:href. I will test with a full build. Kind regards, Ben. On 15/07/14 14:11, Angreani, Rini (Mineral Resources, Kensington) wrote: It's also done in gml2/GMLEncodingUtils.GeometryPropertyType_getProperty. Cheers Rini -Original Message- From: Caradoc-Davies, Ben (Mineral Resources, Kensington) Sent: Tuesday, 15 July 2014 12:25 PM To: Andrea Aime Cc: Geotools-Devel list; Angreani, Rini (Mineral Resources, Kensington) Subject: Re: [Geotools-devel] App-schema, xlinks, and curves Andrea, I am investigating now. My understanding is that the correct behaviour is to encode no content as this property is being mapped by reference. The handling of automatic (duplicate id) href is in ComplexSupportXSAnyTypeBinding and its neighbours, but there may be assumptions based on the behaviour of other bindings. Content is suppressed by setting the children to an empty container. I will have to check to see where mapped href is set differently and what changes there are to the geometry bindings. Kind regards, Ben. On 11/07/14 18:51, Andrea Aime wrote: Hi all, the arcs pull request has been updated to handle GML encoding of arcs (see the other mail about this topic for details). Generally speaking it's working well, GeoServer builds also work fine minus one hitch in the app-schema tests Basically, I have a xlink test failing on me, XLinkGeometryTest.testGeometry, that ends up encoding something like this: ex:multisurface xlink:href=#geom10/ ex:multipolygon xlink:href=#geom11/ ex:geometryref xlink:href=xlinkvalue1/ ex:curveref xlink:href=xlinkvalue2 gml:LineString gml:posList/ /gml:LineString /ex:curveref ... In other words, the other elements are empty, but curve is not. However, I don't see where the href management is, and how it's supposed to behave. Looking at the results, it seems the encoder was fed with an empty linestring... Do you have any quick pointers? Cheers Andrea -- == GeoServer Professional Services from the experts! Visit http://goo.gl/NWWaa2 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 --- -- Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
Re: [Geotools-devel] App-schema, xlinks, and curves
Andrea, here is a pull request for your feature branch: https://github.com/aaime/geotools/pull/5 Full builds of this branch and GeoServer master both pass for me. Kind regards, Ben. On 15/07/14 14:19, Ben Caradoc-Davies wrote: Thanks, Rini. I think I found the problem: there is an unwanted @Override in org.geotools.gml3.bindings.ext.CurvePropertyTypeBinding that prematurely returns the geometry object before the superclass has a chance to recognise that it should be nulled out because there is an xlink:href. I will test with a full build. Kind regards, Ben. On 15/07/14 14:11, Angreani, Rini (Mineral Resources, Kensington) wrote: It's also done in gml2/GMLEncodingUtils.GeometryPropertyType_getProperty. Cheers Rini -Original Message- From: Caradoc-Davies, Ben (Mineral Resources, Kensington) Sent: Tuesday, 15 July 2014 12:25 PM To: Andrea Aime Cc: Geotools-Devel list; Angreani, Rini (Mineral Resources, Kensington) Subject: Re: [Geotools-devel] App-schema, xlinks, and curves Andrea, I am investigating now. My understanding is that the correct behaviour is to encode no content as this property is being mapped by reference. The handling of automatic (duplicate id) href is in ComplexSupportXSAnyTypeBinding and its neighbours, but there may be assumptions based on the behaviour of other bindings. Content is suppressed by setting the children to an empty container. I will have to check to see where mapped href is set differently and what changes there are to the geometry bindings. Kind regards, Ben. On 11/07/14 18:51, Andrea Aime wrote: Hi all, the arcs pull request has been updated to handle GML encoding of arcs (see the other mail about this topic for details). Generally speaking it's working well, GeoServer builds also work fine minus one hitch in the app-schema tests Basically, I have a xlink test failing on me, XLinkGeometryTest.testGeometry, that ends up encoding something like this: ex:multisurface xlink:href=#geom10/ ex:multipolygon xlink:href=#geom11/ ex:geometryref xlink:href=xlinkvalue1/ ex:curveref xlink:href=xlinkvalue2 gml:LineString gml:posList/ /gml:LineString /ex:curveref ... In other words, the other elements are empty, but curve is not. However, I don't see where the href management is, and how it's supposed to behave. Looking at the results, it seems the encoder was fed with an empty linestring... Do you have any quick pointers? Cheers Andrea -- == GeoServer Professional Services from the experts! Visit http://goo.gl/NWWaa2 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 --- -- Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
Re: [Geotools-devel] App-schema, xlinks, and curves
Thanks, Rini, your advice was accurate: the broken override I removed was masking the functionality in this method. Kind regards, Ben. On 15/07/14 14:11, Angreani, Rini (Mineral Resources, Kensington) wrote: It's also done in gml2/GMLEncodingUtils.GeometryPropertyType_getProperty. -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
Re: [Geotools-devel] App-schema, xlinks, and curves
Andrea, I am investigating now. My understanding is that the correct behaviour is to encode no content as this property is being mapped by reference. The handling of automatic (duplicate id) href is in ComplexSupportXSAnyTypeBinding and its neighbours, but there may be assumptions based on the behaviour of other bindings. Content is suppressed by setting the children to an empty container. I will have to check to see where mapped href is set differently and what changes there are to the geometry bindings. Kind regards, Ben. On 11/07/14 18:51, Andrea Aime wrote: Hi all, the arcs pull request has been updated to handle GML encoding of arcs (see the other mail about this topic for details). Generally speaking it's working well, GeoServer builds also work fine minus one hitch in the app-schema tests Basically, I have a xlink test failing on me, XLinkGeometryTest.testGeometry, that ends up encoding something like this: ex:multisurface xlink:href=#geom10/ ex:multipolygon xlink:href=#geom11/ ex:geometryref xlink:href=xlinkvalue1/ ex:curveref xlink:href=xlinkvalue2 gml:LineString gml:posList/ /gml:LineString /ex:curveref ... In other words, the other elements are empty, but curve is not. However, I don't see where the href management is, and how it's supposed to behave. Looking at the results, it seems the encoder was fed with an empty linestring... Do you have any quick pointers? Cheers Andrea -- == GeoServer Professional Services from the experts! Visit http://goo.gl/NWWaa2 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 --- -- Open source business process management suite built on Java and Eclipse Turn processes into business applications with Bonita BPM Community Edition Quickly connect people, data, and systems into organized workflows Winner of BOSSIE, CODIE, OW2 and Gartner awards http://p.sf.net/sfu/Bonitasoft ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Mineral Resources Flagship Australian Resources Research Centre -- Want fast and easy access to all the code in your enterprise? Index and search up to 200,000 lines of code with a free copy of Black Duck Code Sight - the same software that powers the world's largest code search on Ohloh, the Black Duck Open Hub! Try it now. http://p.sf.net/sfu/bds ___ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel