wkt icon indicating copy to clipboard operation
wkt copied to clipboard

Support parse POINTZ, POINTM and POINTZM

Open ariesdevil opened this issue 1 year ago • 4 comments

  • [x] I agree to follow the project's code of conduct.
  • [ ] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.
  • [x] I ran cargo fmt

ariesdevil avatar Feb 07 '24 09:02 ariesdevil

I guess this will make the conversion to geo-types lossy. I don't think Z/M are available there, so there's not much you can do, just wanted to point that out.

lnicola avatar Feb 07 '24 11:02 lnicola

I guess this will make the conversion to geo-types lossy. I don't think Z/M are available there, so there's not much you can do, just wanted to point that out.↳

Thanks point it out, I see Coord in geo-types only support x and y, I think it needs to be refactored later.

ariesdevil avatar Feb 07 '24 11:02 ariesdevil

Fixed, PTAL again.

ariesdevil avatar Feb 07 '24 11:02 ariesdevil

You didn't add that comment about matches!, but whatever 😀.↳

Sorry, I forgot it🤔

ariesdevil avatar Feb 07 '24 11:02 ariesdevil

We've misspelled POINT Z, POINT M, and POINT ZM. See https://github.com/georust/wkt/issues/114

Can you take a look @ariesdevil?

Otherwise I think we should revert this.

michaelkirk avatar Jul 17 '24 17:07 michaelkirk