[Geotools-devel] uomLabels in gml 3.2

2013-07-17 Thread Rini.Angreani
Hi Justin, I am trying to get uomLabels to appear in LineString, based on GML 3.2: http://schemas.opengis.net/gml/3.2.1/geometryBasic0d1d.xsd. (Actually from GML 3.1.0 onwards). I can see if I make changes to AbstractGeometryTypeBinding, it will create the same problem with srsDimension (where

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Robin Pradel
Hi developers with permission to accept/apply pull requests, Can somebody have a look at the pull requests mentioned below? These ones solve the issue about synonyms in databases http://jira.codehaus.org/browse/GEOT-4486 Andrea had a look at the code in a previous pull request which was ok with

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Jody Garnett
I had a look, but have a hesitation accepting a pull request when I am not in position to test (against oracle for SYNONYM). I have just finished my pre-flight checks for GeoTools, but would like to see if we can get this in since you have asked so nicely. While we wait to see if an active

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Ben Caradoc-Davies
I am running the jdbc-oracle online tests on master against Oracle 11g 11.2.0.3.0. Don't you think it is a little close to release to squeeze this one in? Kind regards, Ben. On 17/07/13 16:09, Jody Garnett wrote: I had a look, but have a hesitation accepting a pull request when I am not in

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Ben Caradoc-Davies
Jody, it took twice as long a normal to run and failed with a NPE locally: OracleTypeNamesTestOnlineTestCase.run:123-JDBCTypeNamesTest.testTypeNames:36 ยป NullPointer And on Jenkins: https://cgsrv8.arrc.csiro.au/jenkins/view/geoserver-master/job/geotools-master/529/consoleText Kind regards,

[Geotools-devel] [jira] (GEOT-4512) Build failure in gt-jdbc-oracle OracleTypeNamesTest

2013-07-17 Thread Ben Caradoc-Davies (JIRA)
Ben

[Geotools-devel] Oracle tests skipped on Hudson

2013-07-17 Thread Ben Caradoc-Davies
geotools-master-jdbc Oracle tests are being skipped on Hudson because of a failure to connect: http://hudson.opengeo.org/hudson/job/geotools-master-jdbc/343/console -- Ben Caradoc-Davies ben.caradoc-dav...@csiro.au Software Engineer CSIRO Earth Science and Resource Engineering Australian

[Geotools-devel] GeoTools 10-beta available for testing (and GeoWebCache release)

2013-07-17 Thread Jody Garnett
The release is done but not announced. If I can ask for any volunteers to test http://gridlock.opengeo.org/geotools/release/10-beta/ The next step is Kevin releasing a GWC (tomorrow?), and then I can continue with the GeoServer 2.4 beta. This means there is a small timeframe where when any

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Jody Garnett
Well that is sadly terrible :( I will rollback the change and start my release story again. -- Jody Garnett On Wednesday, 17 July 2013 at 7:19 PM, Ben Caradoc-Davies wrote: Jody, it took twice as long a normal to run and failed with a NPE locally:

Re: [Geotools-devel] GeoTools 10-beta available for testing (and GeoWebCache release)

2013-07-17 Thread Jody Garnett
Sorry Kevin I need to try again, I suffered a lapse of judgement and accepted a pull request without being in position to test it locally :( -- Jody Garnett On Wednesday, 17 July 2013 at 8:43 PM, Jody Garnett wrote: The release is done but not announced. If I can ask for any

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Robin Pradel
Ok, but I think this is not really necessary. Only somebody has to grant the permission for synonyms to database access of the test run, or am I wrong? Regards, Robin Von: Jody Garnett [mailto:jody.garn...@gmail.com] Gesendet: Mittwoch, 17. Juli 2013 12:46 An: Ben Caradoc-Davies Cc: Robin

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Jody Garnett
It is only necessary as Kevin is waiting on me before releasing GWC. I am not sure about the permissions of the test database, most of our tests create the tables and so on as needed. Perhaps Ben will be able to give you more information. -- Jody Garnett On Wednesday, 17 July 2013 at 8:55

[Geotools-devel] Build breaks on JDK7 in module NetCDF Gridc Coverage module

2013-07-17 Thread Brett Walker
Hi Andrea, I have found a build break when done using JDK7. The build breaks in Coverage Experiment/NetCDF Grid Coverage Module. This is caused by a subtle change in the definition of TreeMap#put(K key, V value) in Java 7 when compared in Java 6. Look at the source code and follow the code in

[Geotools-devel] 10-beta ready for GWC (and testing)

2013-07-17 Thread Jody Garnett
It has successfully deployed to maven so GWC should be good to go - let me know when GWC is deployed into maven. If anyone has a moment to test please download from http://gridlock.opengeo.org/geotools/release/10-beta/ I note that the rsync to source forge failed - Indeed it is looking for a

Re: [Geotools-devel] Build breaks on JDK7 in module NetCDF Gridc Coverage module

2013-07-17 Thread Brett Walker
More investigation... Having a look in org.geotools.imageio.unidata.UnidataVariableAdapter; the cause of the problem revolves around the definition and usage of globalDomainExtent (line 278). This is used to store DateRanges and NumberRanges. Each range has a comparator defined in

Re: [Geotools-devel] 10-beta ready for GWC (and testing)

2013-07-17 Thread Ian Turton
would adding -p to that mkdir fix these problems or is the whole directory name wrong? Ian On 17 July 2013 15:09, Jody Garnett jody.garn...@gmail.com wrote: It has successfully deployed to maven so GWC should be good to go - let me know when GWC is deployed into maven. If anyone has a

Re: [Geotools-devel] Build breaks on JDK7 in module NetCDF Gridc Coverage module

2013-07-17 Thread Brett Walker
Thanks for the response. From recent commits, it seems that you added the org.geotools.imageio.unidata.UnidataVariableAdapter class on the 3/7 (b45cf97, Improve the NG raster API, add support in NetCDF readers for multiple coverages, read only structured grid coverage reader support, high

Re: [Geotools-devel] Build breaks on JDK7 in module NetCDF Gridc Coverage module

2013-07-17 Thread Andrea Aime
On Wed, Jul 17, 2013 at 4:30 PM, Brett Walker brett.wal...@geometryit.comwrote: More investigation... Having a look in org.geotools.imageio.unidata.UnidataVariableAdapter; the cause of the problem revolves around the definition and usage of globalDomainExtent (line 278). This is used to

[Geotools-devel] [jira] (GEOT-4497) remove use of proprietary com.sun package classes

2013-07-17 Thread Simone Giannecchini (JIRA)
Simone

[Geotools-devel] [jira] (GEOT-4513) Upgrade jaitools dependency to 1.3.1

2013-07-17 Thread Simone Giannecchini (JIRA)
Simone

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Simone Giannecchini
Ciao Bret, I had a quick look and I am happy withe changes. I will create a jira and backport to 9.x Regards, Simone Giannecchini == Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information. == Ing. Simone Giannecchini @simogeo Founder/Director GeoSolutions S.A.S.

Re: [Geotools-devel] 10-beta ready for GWC (and testing)

2013-07-17 Thread Jody Garnett
I think it may be that the regex does not expect beta and is thus calculating the parent name wrong. I would of expected g/ge/geotools/GeoTools 10/10-beta In any case it is not a trouble, and I have some time while we wait for GWC to be released. -- Jody Garnett On Thursday, 18 July 2013

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Jody Garnett
Robin and Ben: If you can clean up this patch today, and Ben verifies I can regenerate the release. It looks like we may get a last minuet fix for Java 7 (based on the other email thread). -- Jody Garnett On Wednesday, 17 July 2013 at 9:40 PM, Jody Garnett wrote: It is only necessary

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Jody Garnett
Were you expecting this in the current release? If so please advise. (And thanks for stepping up with all the pull request Brett) -- Jody Garnett On Thursday, 18 July 2013 at 2:58 AM, Simone Giannecchini wrote: Ciao Bret, I had a quick look and I am happy withe changes. I will create a

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Brett Walker
I have no need for it to be in the 10 release. I did find a bug when building using jdk7 on a different matter altogether. I have talked to Andrea about this. Brett Sent from my iPad On 18/07/2013, at 9:52 AM, Jody Garnett jody.garn...@gmail.commailto:jody.garn...@gmail.com wrote: Were you

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Michael Bedward
Just poking my nose in here although it's not really my business... The 1.3.1 version contains a number of bug fixes, including important contributions from Simone, so it would be nice to move up to it as soon as possible. Michael On 18 July 2013 09:58, Brett Walker brett.wal...@geometryit.com

Re: [Geotools-devel] Preparing for the release

2013-07-17 Thread Ben Caradoc-Davies
No it is all good. Not a GeoTools bug. Resolved by granting CREATE SYNONYM privilege to the Oracle test user. https://jira.codehaus.org/browse/GEOT-4512 On 17/07/13 18:46, Jody Garnett wrote: Well that is sadly terrible :( I will rollback the change and start my release story again. --

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Ben Caradoc-Davies
Note that it breaks GeoServer wps-core on master and 2.3.x: Build failure in wps-core CoverageUtilities caused by jaitools upgrade https://jira.codehaus.org/browse/GEOS-5892 On 18/07/13 08:41, Michael Bedward wrote: Just poking my nose in here although it's not really my business... The 1.3.1

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Michael Bedward
It was GeoSolutions that urgently wanted the bug fixes to JAITools that are contained in v1.3.1. That was a few months ago, so it seems surprising that neither GeoTools or GeoServer have taken advantage of them. If that's due to any continuing problems at the JAITools end, let me know. Michael

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Ben Caradoc-Davies
There is an API change. Looks like wps-core needs to use a builder rather than directly invoking a constructor. Generics might also have been added. On 18/07/13 11:55, Michael Bedward wrote: It was GeoSolutions that urgently wanted the bug fixes to JAITools that are contained in v1.3.1. That

[Geotools-devel] [jira] (GEOT-4514) GeoJSON module cannot parse geojson objects with null geometries

2013-07-17 Thread Gerson Galang (JIRA)
Gerson

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Brett Walker
Hi Ben, I noticed the API change. It lost the ability to supply a default value. I wasn't impressed with it. If you look at my patch to help with the upgrade it might give you some ideas in how to address the GeoServer problems. As I said in the original post; I ' brutalise a method' to get

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Michael Bedward
On 18 July 2013 14:43, Brett Walker brett.wal...@geometryit.com wrote: It lost the ability to supply a default value. I wasn't impressed with it. Concerning GS - I know nothing :) Concerning JAITools and your not being impressed - I suggest you discuss any concerns with Simone since he

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Ian Turton
I was going to try to start the GS release today but I'll hold until later to see if anyone want's to fix this as the european day progresses. Ian On 18 July 2013 04:39, Ben Caradoc-Davies ben.caradoc-dav...@csiro.auwrote: Note that it breaks GeoServer wps-core on master and 2.3.x: Build

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Brett Walker
He must be fine with it. He merged my pull request and back-ported it to the 9.x branch The way I seemed to fixed the default value is, and this is my point of uncertainty, can be seen in the following code extract: for (int i = 0; i size; i++) { final Range range =

Re: [Geotools-devel] FW: Pull Request to upgrade JAITools

2013-07-17 Thread Andrea Aime
On Thu, Jul 18, 2013 at 5:55 AM, Michael Bedward michael.bedw...@gmail.comwrote: It was GeoSolutions that urgently wanted the bug fixes to JAITools that are contained in v1.3.1. That was a few months ago, so it seems surprising that neither GeoTools or GeoServer have taken advantage of them.