[Geotools-devel] Second question and code review request

2007-06-19 Thread Jody Garnett
Question number two here; and a code review request. The following two test cases: - DefaultFactoryOnlineTest - in epsg/oracle - DefaultFactoryTest - in epsg/hsql Have been giving me problems when they perform an: - assertSame(((Conversion) operation).getMethod(), ((Conversion) projection).getMe

Re: [Geotools-devel] My first GeoTools validation questions...

2007-06-19 Thread Sunburned Surveyor
Jody wrote: "Hey we fell off the development list again - that is not cool." My bad. I keep forgetting to hit the "reply to all" button. Jody wrote: "Call it a FeatureCollection then. I seem to recall Jump having a FeatureType - indeed I reviewed it as a candidate for GeoTools." JUMP has Feature

[Geotools-devel] Speaking of failed builds - 2.2.x does not build right now because of a GML test

2007-06-19 Thread Jody Garnett
Could I ask that a build box be set to watch this branch? Please? It is not very kind to ask Richard to work without a safety net. Here are the details: org.geotools.data.gml.GMLDataStoreTest > java.io.IOException: java.net.ConnectException: Connection refused: > connect > at org.geotool

Re: [Geotools-devel] My first GeoTools validation questions...

2007-06-19 Thread Jody Garnett
Hey we fell off the development list again - that is not cool. Sunburned Surveyor wrote: > This might be a little bit of a problem. OpenJUMP doesn't specify > FeatureTypes. Call it a FeatureCollection then. I seem to recall Jump having a FeatureType - indeed I reviewed it as a candidate for GeoT

[Geotools-devel] [continuum] BUILD SUCCESSFUL: Geotools 2.3.x

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/86/buildId/1040 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

Re: [Geotools-devel] ArcSDE build fixed, methods added to dummy api

2007-06-19 Thread Gabriel Roldán
Hi Saul, sorry I missed you on the #geosever channel. I guess we could do the same as for the other fields, to replace the switch statements by if's...? gonna take a look at it tomorrow morning, its quite late here already. regards and thanks Gabriel On Tuesday 19 June 2007 23:48:41 Farber,

[Geotools-devel] [jira] Created: (GEOT-1348) PostGIS createSchema method fails when asked to represent a String.class

2007-06-19 Thread Jody Garnett (JIRA)
PostGIS createSchema method fails when asked to represent a String.class Key: GEOT-1348 URL: http://jira.codehaus.org/browse/GEOT-1348 Project: GeoTools Issue Type: Bug

[Geotools-devel] ArcSDE build fixed, methods added to dummy api

2007-06-19 Thread Farber, Saul (EEA)
Hey gabriel, I added the sde dummy methods required for the latest ArcSDE trunk changes. I had to make some 'public static final int' fields, though (due to their use in case: statements) so that adds another blocker berween binary compatibility between the dummy-compiled datastore jar and th

[Geotools-devel] [continuum] BUILD SUCCESSFUL: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1035 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

Re: [Geotools-devel] 3 geometry bugs I'm looking into

2007-06-19 Thread Graham Davis
Jackson, I'm still working on the relate() bug. I've fixed some of the bug (a few JTS tests were failing with this operation, and I've fixed one instance at least). Have you had a chance to look at the other two bugs? Graham. Graham Davis wrote: >Jackson, > >Glad to hear from you, and it w

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1034 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] Build is fixed today

2007-06-19 Thread Jody Garnett
Well sorry about that guys; between - ArcSDE hating us - Richard committing his url fix across 2.2, 2.3 and trunk ... - Me tripping up the Factory SPI lookup for DefaultFactory The build was broken for a bit... it is back now. Jody > Opps - am breaking the build RIGHT NOW in order to do just that

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1033 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1032 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1031 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] Referencing META-INF/services question

2007-06-19 Thread Jody Garnett
Looking at the META-INF/services folder for main I noticed the following: org.geotools.referencing.factory.epsg.DefaultFactory In my understanding DefaultFactory is an superclass for others to extend? Should it really be registered here (can it even do anything on its own?) Perhaps the intentio

Re: [Geotools-devel] My first GeoTools validation questions...

2007-06-19 Thread Jody Garnett
Just checked - on trunk it uses FeatureCollection. public void runFeatureTests(String dsID, FeatureCollection collection, ValidationResults results); ValidationResults operates as a visitor in a manner similar to TestResults in JUnit. have fun, Jody > I've got some good news. I think I can ma

Re: [Geotools-devel] My first GeoTools validation questions...

2007-06-19 Thread Jody Garnett
> I've got some good news. I think I can make use of the validation > framework in some of my geomerty and spatial relationship validation > work. > cool. There are lots of formal docs if you want them - talking about what the files are and what operations were needed to be complete for the f

Re: [Geotools-devel] change to SimpleTypeBuilder: first experiment

2007-06-19 Thread Justin Deoliveira
Yeah i have noticed the same, and it keeps biting me too. I am all for changing it and have been meaning to do so just have not gotten around to it. Fly at it if you want to Gabriel. -Justin Gabriel Roldán wrote: > Hi, > > trying out a change to SimpleTypeBuilder it turns out that namespace is

[Geotools-devel] My first GeoTools validation questions...

2007-06-19 Thread Sunburned Surveyor
I've got some good news. I think I can make use of the validation framework in some of my geomerty and spatial relationship validation work. I've also got some bad news. I'm running into the FeatureModel problem again. :] I think there are two approaches I could take: [1] I can convert all of my

[Geotools-devel] [continuum] BUILD FAILURE: Geotools 2.3.x

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/86/buildId/1030 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1029 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] change to SimpleTypeBuilder: first experiment

2007-06-19 Thread Gabriel Roldán
Hi, trying out a change to SimpleTypeBuilder it turns out that namespace is mandatory (i.e. DefaultFeatureType constructor throws NPE is the namespace in the passed in TypeName is null). Is this the intender behaiour? Gabriel --

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1028 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] [continuum] BUILD SUCCESSFUL: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1027 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] arcsde out of the build --- until you fix it :-)

2007-06-19 Thread Jody Garnett
Morning :-D Today a bunch of us have been unable to work because of a failure in the arcsde module... > C:\java\geotools\trunk\modules\unsupported\arcsde\datastore\src\main\java\org\geotools\arcsde\data\ArcSDEAdapter.java:[2 > 94,21] cannot find symbol > symbol : method prepareQueryInfo(com.esri

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1026 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

Re: [Geotools-devel] Auto-commit transaction in *ShapeFileDatastore in GT 2.2.x

2007-06-19 Thread Jody Garnett
We should commit it in the source folder of data; so it can be reused by the datastore implementators. This idea (a set of common conformance tests) has a jira issue somewhere already. Good bug hunting; I am afraid at the time the DataStores were written we did not have a user interface around

Re: [Geotools-devel] Auto-commit transaction in *ShapeFileDatastore in GT 2.2.x

2007-06-19 Thread Vitali Diatchkov
What is an appropriate place to commit FeatureEventsTest that tests FeatureStore interface of ShapefileDataStore for FeatureEvents fired by notification mechanism of FeatureListenersManager for autocommit and non-autocommit transactions? The autocommit transaction case fails right now because no o

Re: [Geotools-devel] Class renaming in referencing module: is "svn move" used?

2007-06-19 Thread Jody Garnett
Martin Desruisseaux wrote: > Jody Garnett a écrit : >> But the old file is still there (and in use?) > It doesn't matter; many files can share common history. And this file > will be removed soon, so the history will be lost if we didn't used > "svn copy". Understood; I am backing out my changes

Re: [Geotools-devel] Build to be broken tomorrow

2007-06-19 Thread Jody Garnett
Opps - am breaking the build RIGHT NOW in order to do just that :-( Jody > Tomorrow morning (European time), I will break Geotools build for a few > minutes > in order to repair some history lost in classes renaming. I will: > > 1) Temporarily delete ReferencingObjectFactory, CanonicalSet >

Re: [Geotools-devel] Minor notice about GeoTools.getDefaultHints()

2007-06-19 Thread Jody Garnett
Martin Desruisseaux wrote: > Jody Garnett a écrit : >> Oh great - I tried to make it do that myself (see "trouble with >> hints" message thread from a couple months ago) and got stuck. >> If this is going to be a general pattern we should include it in our >> instructions on how to write a Factor

[Geotools-devel] [continuum] BUILD FAILURE: Geotools 2.3.x

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/86/buildId/1025 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

[Geotools-devel] Build to be broken tomorrow

2007-06-19 Thread Martin Desruisseaux
Tomorrow morning (European time), I will break Geotools build for a few minutes in order to repair some history lost in classes renaming. I will: 1) Temporarily delete ReferencingObjectFactory, CanonicalSet and some other new classes. 2) Commit the deletion (build broken here) 3) Rei

Re: [Geotools-devel] Class renaming in referencing module: is "svn move" used?

2007-06-19 Thread Martin Desruisseaux
Jody Garnett a écrit : > But the old file is still there (and in use?) It doesn't matter; many files can share common history. And this file will be removed soon, so the history will be lost if we didn't used "svn copy". > If I understand you I should ... > # svn mv old new > # svn add old (mak

Re: [Geotools-devel] Minor notice about GeoTools.getDefaultHints()

2007-06-19 Thread Martin Desruisseaux
Jody Garnett a écrit : > Oh great - I tried to make it do that myself (see "trouble with hints" > message thread from a couple months ago) and got stuck. > If this is going to be a general pattern we should include it in our > instructions on how to write a FactoryFinder (so CommonFactoryFinder

Re: [Geotools-devel] Converting java.net.URLs to java.io.Files

2007-06-19 Thread Jody Garnett
Richard Gould wrote: > I'm sticking it in DataUtilities and I will add this to the coding > conventions. > > On the GeoTools wiki: > http://docs.codehaus.org/display/GEOT/5.1.6+Converting+URLs+to+Files > And I created a section on the uDig Wiki for Coding Conventions: > http://udig.refractions.n

Re: [Geotools-devel] Class renaming in referencing module: is "svn move" used?

2007-06-19 Thread Jody Garnett
Martin Desruisseaux wrote: > Jody Garnett a écrit : >> I believe it is because the origional FactoryGroup is still there >> (and has the history). I am doing the create a new class, deprecate >> the old one, and making the old one extend the new one routine. > You should use "svn copy". This is v

Re: [Geotools-devel] Class renaming in referencing module: is "svn move" used?

2007-06-19 Thread Martin Desruisseaux
Jody Garnett a écrit : > I believe it is because the origional FactoryGroup is still there (and > has the history). I am doing the create a new class, deprecate the old > one, and making the old one extend the new one routine. You should use "svn copy". This is very important in order to keep th

Re: [Geotools-devel] Converting java.net.URLs to java.io.Files

2007-06-19 Thread Richard Gould
I'm sticking it in DataUtilities and I will add this to the coding conventions. On the GeoTools wiki: http://docs.codehaus.org/display/GEOT/5.1.6+Converting+URLs+to+Files And I created a section on the uDig Wiki for Coding Conventions: http://udig.refractions.net/confluence/display/DEV/5.+Coding

Re: [Geotools-devel] Class renaming in referencing module: is "svn move" used?

2007-06-19 Thread Jody Garnett
I believe it is because the origional FactoryGroup is still there (and has the history). I am doing the create a new class, deprecate the old one, and making the old one extend the new one routine. Cheers, Jody Martin Desruisseaux wrote: > Why the following: > > svn log > modules/library/refer

Re: [Geotools-devel] Minor notice about GeoTools.getDefaultHints()

2007-06-19 Thread Jody Garnett
Oh great - I tried to make it do that myself (see "trouble with hints" message thread from a couple months ago) and got stuck. If this is going to be a general pattern we should include it in our instructions on how to write a FactoryFinder (so CommonFactoryFinder works in the same manner). Can

[Geotools-devel] [continuum] BUILD FAILURE: Geotools

2007-06-19 Thread Continuum
http://geo.openplans.org:9090/continuum/servlet/continuum/target/ProjectBuild.vm/view/ProjectBuild/id/1/buildId/1023 - This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and

Re: [Geotools-devel] Code in GeoTools?

2007-06-19 Thread Jody Garnett
Sunburned Surveyor wrote: > I had posted some ideas for collaboration opportunities to the GeoAPI > list. I realized after some discussion on that thread that GeoAPI > wasn't the best vehicle for the type of collaboration I was > discussing. > > The two ideas I mentioned was geometry and spatial re

Re: [Geotools-devel] GSoC weekly report

2007-06-19 Thread Jan Jezek
Here is whole stack trace: org.opengis.parameter.InvalidParameterValueException: Argument "info" should not be null. at org.geotools.referencing.AbstractIdentifiedObject.ensureNonNull(AbstractIdentifiedObject.java:1026) at org.geotools.referencing.Properties.(Properties.java:63)

[Geotools-devel] Class renaming in referencing module: is "svn move" used?

2007-06-19 Thread Martin Desruisseaux
Why the following: svn log modules/library/referencing/src/main/java/org/geotools/referencing/factory/ReferencingObjectFactory.java give me no history? Do I'm missing some SVN switch or did you performed the class renaming with something else than "svn move"? Please don't rely too much on SubE

Re: [Geotools-devel] GSoC weekly report

2007-06-19 Thread Martin Desruisseaux
Jan Jezek a écrit : > - and then when trying to make new DefaultOperationMethod(trans) I'm getting > strange exception: > > org.opengis.parameter.InvalidParameterValueException: Argument "info" should > not be null. > at > org.geotools.referencing.AbstractIdentifiedObject.ensureNonNull(Ab

[Geotools-devel] [jira] Created: (GEOT-1347) JbdcTextFeatureWriter quotes numbers, making them look like strings to Oracle

2007-06-19 Thread Andrea Aime (JIRA)
JbdcTextFeatureWriter quotes numbers, making them look like strings to Oracle - Key: GEOT-1347 URL: http://jira.codehaus.org/browse/GEOT-1347 Project: GeoTools Issue

[Geotools-devel] GSoC weekly report

2007-06-19 Thread Jan Jezek
Hi Jesse, Martin and all, As we have agreed I'm sending my weekly status updates. I've started to implement the GridBuilder as we discussed with Martin (I sent the copy to [EMAIL PROTECTED] as well) and I don't see any problems there. Before I'll produce this builder I've tried to prepare test/

Re: [Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Andrea Antonello
Sure, I just added it. Andrea Martin Desruisseaux probaly wrote: > Andrea Antonello a écrit : >> According to the above, this is optimized for Sicily even if the last >> parameter is different between yours and the one GRASS supplies: >> rome40 "towgs84=-104.1,-49.1,-9.9,0.971,-2.917,0.714,-11

Re: [Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Martin Desruisseaux
Andrea Antonello a écrit : > According to the above, this is optimized for Sicily even if the last > parameter is different between yours and the one GRASS supplies: > rome40 "towgs84=-104.1,-49.1,-9.9,0.971,-2.917,0.714,-11.68" "Italy > (Peninsular Part)" "Accuracy 3-4m" > rome40 "towgs84=-168.6

[Geotools-devel] Minor notice about GeoTools.getDefaultHints()

2007-06-19 Thread Martin Desruisseaux
Hello Jody Thanks for the update to the referencing module. Just a minor notice. There is no need to replace code like: ReferencingFactoryFinder.getCRSFactory(null) by ReferencingFactoryFinder.getCRSFactory(GeoTools.getDefaultHints()) because default hints are automatically taken in

[Geotools-devel] [jira] Created: (GEOT-1346) ReprojectFetureReader fails to close inner readers

2007-06-19 Thread Andrea Aime (JIRA)
ReprojectFetureReader fails to close inner readers -- Key: GEOT-1346 URL: http://jira.codehaus.org/browse/GEOT-1346 Project: GeoTools Issue Type: Bug Components: data Affects Vers

[Geotools-devel] [jira] Created: (GEOT-1345) ReprojectFeatureResults NPEs when null geometries are found

2007-06-19 Thread Andrea Aime (JIRA)
ReprojectFeatureResults NPEs when null geometries are found --- Key: GEOT-1345 URL: http://jira.codehaus.org/browse/GEOT-1345 Project: GeoTools Issue Type: Bug Components: dat

Re: [Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Andrea Antonello
Thanks for the clear explanations Andrea, now I got the points and have everything I need to move this all to the udig list. Geotools has everything defined, so I think what's left out is a part of the behavioural part in Udig. I'll wait for the UDig list to react to this. Ciao Andrea Andrea A

Re: [Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Andrea Aime
Andrea Antonello ha scritto: > Andrea Aime probaly wrote: > I don't understand you exactly. When you use reprojection for calculus > and not just for visualization, things can get bad if the system does > what it wants without asking. So when I create a map from a shapefile, > why shouldn't the sys

Re: [Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Andrea Antonello
Andrea Aime probaly wrote: > Andrea Antonello ha scritto: >> Hi, I send this to both geotools and udgi, since I assume this is a >> geotools issue, but I have problems with the udig behaviour regarding >> this. >> >> When I try to reproject from (example) WGS84 4326 to Gauss Boaga 3003 I >> get an

Re: [Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Andrea Aime
Andrea Antonello ha scritto: > Hi, I send this to both geotools and udgi, since I assume this is a > geotools issue, but I have problems with the udig behaviour regarding this. > > When I try to reproject from (example) WGS84 4326 to Gauss Boaga 3003 I > get an exception for the missing bursa wolf

[Geotools-devel] how to deal with bursa wolf?

2007-06-19 Thread Andrea Antonello
Hi, I send this to both geotools and udgi, since I assume this is a geotools issue, but I have problems with the udig behaviour regarding this. When I try to reproject from (example) WGS84 4326 to Gauss Boaga 3003 I get an exception for the missing bursa wolf paramas. If, as I understand, these ar

Re: [Geotools-devel] Code in GeoTools?

2007-06-19 Thread Andrea Aime
Sunburned Surveyor ha scritto: > I had posted some ideas for collaboration opportunities to the GeoAPI > list. I realized after some discussion on that thread that GeoAPI > wasn't the best vehicle for the type of collaboration I was > discussing. > > The two ideas I mentioned was geometry and spat