georust/wkt

Relax ToWkt impls to work with CoordNum

rmanoka opened this issue · 1 comments

rmanoka commented

Currently the ToWkt impls. need T: CoordFloat + Display on the geometry. Can this be relaxed to T: CoordNum + Display? Or, is this a big structural change? Ideally, even FromWkt should rather use CoordNum + FromStr than CoordFloat if that's possible.

michaelkirk commented

I think it should be possible! It'll require some changes to the underlying Wkt type which is currently restricted to num_traits::Float - but I can't imagine why it'd actually be relying on Float behavior for anything.

I can take a look tomorrow.

  • ❤️1