I am also one of the module maintainer (see pom.xml file). And yes 
please hack away on the parser - its main purpose is to give people an 
easy "readable" way to create ISO Geometry content.

I do need to ask for a couple more things:
- issue tracker - you got one already! thanks ...
- documentation page - there is one on the wiki already but it needs 
examples, and I need the PicoContainer example as well ASAP (since the 
WKTParser is the easiest example right now). In both cases cutting and 
pasting from the test cases should do the trick.

GEOT-1355
> In debugging GEOT-1311 <http://jira.codehaus.org/browse/GEOT-1311>, 
> I've come across some WKT tests that are failing because they are 
> creating RingImpl objects from the WKT LINEARRING. Because of this, 
> they are considered Areas when doing Relate() operations, when really 
> they aren't Areas, they should be closed linestrings. The ISO Geometry 
> 19107 specs define Rings as "a single connected component of a 
> GM_SurfaceBoundary". So a Ring should always be used as part of a 
> SurfaceBoundary, and in this case we definitely aren't a surface 
> (polygon).
>
> The parser should be updated to create and return a curve instead of a 
> ring for the WKT LINEARRING.
>
I am a bit confused here - but it is my own ignorance to blame.
- you want to change the parser for "LINEARRING" to create Curve
- the current target is RingImpl which implements Ring which is a 
Surface and that was hardly the aim

Okay that makes sense to me ... but I think we are going to have to 
include pictures in the user docs for everyone who has been sticking 
with SFSQL for years :-)

Jody




Graham Davis wrote:
> Jody,
>
> I see that you are author and maintainer of WKTParser.java in the main 
> module.  I'm working on the GEOT-1311 issue and have come across a 
> change that should be made to the WKTParser.  I've created an JIRA bug 
> to track the issue:  http://jira.codehaus.org/browse/GEOT-1355
>
> Can I make this change and commit it?
>
>   


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to