Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Graham Davis
I see the problem now (the file name has is slightly wrong). Fixing that seems to have fixed the problem! Thanks Martin! Graham. Martin Desruisseaux wrote: > Graham Davis a écrit : > >> Sorry, I DO have that in my files. Is that not how it is suppose to >> be (declaring implementations, no

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Martin Desruisseaux
Graham Davis a écrit : > Sorry, I DO have that in my files. Is that not how it is suppose to be > (declaring implementations, not interfaces)? The filename must be the fully qualified interface name. So the filename should be META-INF/services/org.opengis.geometry.PositionFactory The content o

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Graham Davis
Martin Desruisseaux wrote: > But I noticed that org.geotools.geometry.iso.PositionFactoryImpl is > declared in > META-INF/services/org.geotools.geometry.iso.PositionFactory. Don't you > means META-INF/services/org.opengis.geometry.PositionFactory? This is > the factory that GeometryFactoryFind

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Graham Davis
Sorry, I DO have that in my files. Is that not how it is suppose to be (declaring implementations, not interfaces)? Graham. Graham Davis wrote: > Martin Desruisseaux wrote: > >> But I noticed that org.geotools.geometry.iso.PositionFactoryImpl is >> declared in >> META-INF/services/org.geoto

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Martin Desruisseaux
Graham Davis a écrit : > I've commited my recent changes, you can see them in: > > - org.geotools.geometry.iso.PositionFactoryImpl The constructor implementation seems good. But I noticed that org.geotools.geometry.iso.PositionFactoryImpl is declared in META-INF/services/org.geotools.geometry.i

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Graham Davis
Martin Desruisseaux wrote: > > ??? I'm not sure to understand what you means. We don't ignore user > provided hints. We just pick up the one that are relevant to the factory. > > Martin > I think the issue is we don't see where the hints get used to find or create a factory. Going through

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Martin Desruisseaux
Jody Garnett a écrit : > So that is what is supposed to throw the FactoryNotFoundException > exception? I hate seeing exceptions used for flow of control? Seriously > is that what is expected to happen? Or am I reading the code wrong? This is what is expected to happen. I agree that using except

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Graham Davis
I've implemented Martin's suggestions (in positionfactory, since that is the most basic factory and I'm trying to test with that now to make sure the factory finder stuff works as it should). It still can't find my factory. I've commited my recent changes, you can see them in: - org.geotools.

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Jody Garnett
Martin Desruisseaux wrote: > Jody Garnett a écrit : >> In every case we were either directly returning the object provided >> by the no argument constructor, or we were producing a >> FactoryNotFoundException - at which point the later code would not >> work either (as it needs the no argument c

Re: [Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Martin Desruisseaux
Jody Garnett a écrit : > In every case we were either directly returning the object provided by > the no argument constructor, or we were producing a > FactoryNotFoundException - at which point the later code would not work > either (as it needs the no argument constructor to work so it gets an

Re: [Geotools-devel] ArcSDE on trunk - compile error?

2007-07-30 Thread Farber, Saul (EEA)
Yup. Thanks for fixing this guys. I thought I waited long enough to assure a working build before I headed out, but apparently I forgot something. Sorry for screwing up the build! --saul From: Justin Deoliveira [mailto:[EMAIL PROTECTED] Sent: Fri 7/20/2007

Re: [Geotools-devel] IllegalArgumentException on FeatureCollection

2007-07-30 Thread Farber, Saul (EEA)
Hey guys, I'm just back from vacation. Gertjan, do you have a full stack trace that you could send me? Is your postgis database internet-accessible? It'd be great to be able to try and cause the exception on my end. If not, can you give me a (sample?) pgdump of the dataset? --saul

Re: [Geotools-devel] JDBCDataStores and SQL splitter revision

2007-07-30 Thread Farber, Saul (EEA)
Hey jody and david, I'm just back from a vacation since last Thursday. Can we chat about this at the meeting today? --saul From: Jody Garnett [mailto:[EMAIL PROTECTED] Sent: Wed 7/25/2007 1:01 PM To: Farber, Saul (ENV); David Adler; geotools-devel Subject: J

[Geotools-devel] help with FactoryCreator.getServiceProvider

2007-07-30 Thread Jody Garnett
Hi Martin: The problem I found was with the FactoryCreator.getServiceProvider method - it starts out ... > public Object getServiceProvider(final Class category, > final Filterfilter, > final Hints hints, >

Re: [Geotools-devel] Keeping uDig temporarily on Geotools 2.4

2007-07-30 Thread Jody Garnett
Adrian we will tag, uDig when we switch over to GeoTools 2.5. For the longest time our plan was to release a uDig 1.1.1 against GeoTools 2.3 (but the amount of change in GeoTools 2.3 has scared us a bit). Jody > Adrian Custer wrote: > >> Hey all, >> >> As of revision 26360 uDig can be run on

Re: [Geotools-devel] Keeping uDig temporarily on Geotools 2.4

2007-07-30 Thread Jody Garnett
Adrian Custer wrote: > Hey all, > > As of revision 26360 uDig can be run on eclipse 3.3 using geotools 2.4. > This is a stable configuration for SDK developers so I hope we could > keep some form of this configuration around for a while even if others > want to rush to port uDig to the new feature

Re: [Geotools-devel] Keeping uDig temporarily on Geotools 2.4

2007-07-30 Thread Justin Deoliveira
Adrian Custer wrote: > Hey all, > > As of revision 26360 uDig can be run on eclipse 3.3 using geotools 2.4. > This is a stable configuration for SDK developers so I hope we could > keep some form of this configuration around for a while even if others > want to rush to port uDig to the new feature

Re: [Geotools-devel] factory finder

2007-07-30 Thread Graham Davis
Martin, The code we are using is found in referencing->org.geotools.geometry->GeometryFactoryFinder.java and we have begun adding the Hints constructors to PrimitiveFactoryImpl in the unsupported gometry module. Graham. Martin Desruisseaux wrote: > Jody Garnett a écrit : > >> In my underst

Re: [Geotools-devel] Subversion down?

2007-07-30 Thread Paul Ramsey
Back On 30-Jul-07, at 3:13 AM, Andrea Aime wrote: > Hans Häggström ha scritto: >> Hi, >> I'm trying to commit some changes to my GSoC project, but it seems >> http://svn.geotools.org/ is timing out or hanging. Pinging the host >> works, so perhaps the svn server is having some problems? This >>

[Geotools-devel] [jira] Created: (GEOT-1416) Use SDO_TUNE.EXTENT_OF to compute bbox for Oracle data types

2007-07-30 Thread Andrea Aime (JIRA)
Use SDO_TUNE.EXTENT_OF to compute bbox for Oracle data types Key: GEOT-1416 URL: http://jira.codehaus.org/browse/GEOT-1416 Project: GeoTools Issue Type: Bug Components: d

[Geotools-devel] [jira] Created: (GEOT-1415) If the geometry type is specified and it's multi-geom type, simple geoms must be promoted during reading

2007-07-30 Thread Andrea Aime (JIRA)
If the geometry type is specified and it's multi-geom type, simple geoms must be promoted during reading Key: GEOT-1415 URL: http://jira.codehaus.org/browse/G

Re: [Geotools-devel] Keeping uDig temporarily on Geotools 2.4

2007-07-30 Thread Rob Atkinson
Cutting branches to support bug fixes from a stable place makes sense. Adding new functionality from this point is more problematic - the overhead of testing stuff against a branch and the trunk is high, and having the trunk orphaned again is a recipe for frustration and delay. My own thought

Re: [Geotools-devel] Subversion down?

2007-07-30 Thread Andrea Aime
Hans Häggström ha scritto: > Hi, > > I'm trying to commit some changes to my GSoC project, but it seems > http://svn.geotools.org/ is timing out or hanging. Pinging the host > works, so perhaps the svn server is having some problems? This > happened yesterday (or 12 hours ago), and again now. >

Re: [Geotools-devel] Subversion down?

2007-07-30 Thread Adrian Custer
yeah, I'm having trouble as well from the south of france. --adiran On Mon, 2007-07-30 at 13:04 +0300, Hans Häggström wrote: > Hi, > > I'm trying to commit some changes to my GSoC project, but it seems > http://svn.geotools.org/ is timing out or hanging. Pinging the host > works, so perhaps t

[Geotools-devel] Subversion down?

2007-07-30 Thread Hans Häggström
Hi, I'm trying to commit some changes to my GSoC project, but it seems http://svn.geotools.org/ is timing out or hanging. Pinging the host works, so perhaps the svn server is having some problems? This happened yesterday (or 12 hours ago), and again now. I tried to use my svn tools with another

[Geotools-devel] Keeping uDig temporarily on Geotools 2.4

2007-07-30 Thread Adrian Custer
Hey all, As of revision 26360 uDig can be run on eclipse 3.3 using geotools 2.4. This is a stable configuration for SDK developers so I hope we could keep some form of this configuration around for a while even if others want to rush to port uDig to the new feature model. I'd like either for a br