search

jimmyconti / nettopologysuite

Automatically exported from code.google.com/p/nettopologysuite
0 stars 0 forks source link

WKT Z and M flag support #134

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The WKTReader and WKTWriter classes do not support the Z and M flags defined in 
the OGC Simple feature access standard.

At the very least, support should be added to WKTReader.

For example, the following WKT string is supported:

POINT (45.8 29.9 14.8 42.9)

But these strings are not currently supported:

POINT Z (45.8 29.9 14.8)
POINT M (45.8 29.9 42.9)
POINT ZM (45.8 29.9 14.8 42.9)

Original issue reported on code.google.com by simon.ba...@me.com on 26 Nov 2012 at 6:44

GoogleCodeExporter commented 9 years ago 
I have code for that in stock, but imho [i106] has to be straightend prior to 
addressing this.
[i106] is also a blocker for [i129].

Original comment by felix.ob...@netcologne.de on 27 Nov 2012 at 8:13

GoogleCodeExporter commented 9 years ago 
I've checked in a temporary fix for NTS to be able to read XY values for such 
WKT strings (M and Z values are not read, just skipped)

Original comment by peter.lo...@gmail.com on 26 Feb 2013 at 8:16

GoogleCodeExporter commented 9 years ago 
---
For example, the following WKT string is supported:

POINT (45.8 29.9 14.8 42.9)
---

If I use WKTReader::Read() to read the WKT above I get an exeption: "Expected 
word but encountered number". (I downloaded the source of the version 1.13: 
NTS-v1-13-0-SRC.zip in the download section).

Sins the support of the M Coordinate is not really clear for me I would 
recommend one of the following solutions:

1# Skip the M coordinate and at least prevent the Exception
Line 224: if (IsNumberNext(tokens)) GetNextNumber(tokens); // skip M Koordinate

2# Raise an Exception  that sais M-Coordinate is not supported

Option 1# is attached (WKTReader.cs)

Original comment by balz.ste...@gmail.com on 15 Mar 2013 at 10:13

Attachments:

GoogleCodeExporter commented 9 years ago 
The fix is not in the released source, it is in the trunk.
You need some subversion client to grab that.

Original comment by felix.ob...@netcologne.de on 15 Mar 2013 at 10:25

GoogleCodeExporter commented 9 years ago 
can we close this issue?

Original comment by diegogu...@gmail.com on 14 Feb 2014 at 9:48

GoogleCodeExporter commented 9 years ago 
Yes :)

Original comment by balz.ste...@gmail.com on 14 Feb 2014 at 4:32

GoogleCodeExporter commented 9 years ago 
:)

Original comment by diegogu...@gmail.com on 14 Feb 2014 at 5:18