Add CoordinateSequenceFilter to geom.js #402
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm working on making
geo
cross-compatible with Clojurescript in addition to Clojure, which means integrating JSTS alongside JTS, and ran into the same issue as #382.geo
performs all of its reprojection/transformation by implementingCoordinateSequenceFilter
in JTS, and JTS Javadocs seem to suggest that this is the preferred method for doing that kind of transformation.By creating a function that that implements the five methods of
CoordinateSequenceFilter
(and correctly returns the class to be able to satisfyhasInterface
, assuming it's exportable fromgeom.js
), transformations of all geometries seem to be working correctly using the same process here as in JTS.@bjornharrtell