We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WKT strings can store a dimensionality even while being EMPTY. E.g. these are valid:
EMPTY
POINT EMPTY POINT Z EMPTY POINT M EMPTY POINT ZM EMPTY
Currently, the parsed representation does not store this. It only stores a coordinate or a sequence of coordinates, depending on the type:
wkt/src/types/linestring.rs
Line 23 in b2f7399
I think it would be a worthwhile change to also store the dimensionality as part of the geometry. E.g.:
pub struct LineString<T: WktNum> { coords: Vec<Coord<T>>, dim: Dimension, }
This would be nice for two reasons:
dim
Unfortunately, this would be a breaking change because the .0 element of the geometry types is public.
.0
The text was updated successfully, but these errors were encountered:
geo_traits
geojson
No branches or pull requests
WKT strings can store a dimensionality even while being
EMPTY
. E.g. these are valid:Currently, the parsed representation does not store this. It only stores a coordinate or a sequence of coordinates, depending on the type:
wkt/src/types/linestring.rs
Line 23 in b2f7399
I think it would be a worthwhile change to also store the dimensionality as part of the geometry. E.g.:
This would be nice for two reasons:
dim
method for each geometry trait, because it wouldn't have to recurse into the first coordinate.Unfortunately, this would be a breaking change because the
.0
element of the geometry types is public.The text was updated successfully, but these errors were encountered: