Add Z handling to LineSegment and Densifier#658
Add Z handling to LineSegment and Densifier#658mukoki wants to merge 5 commits intolocationtech:masterfrom
Conversation
sync mukoki fork
Update fork
update fork
Signed-off-by: mukoki <m.michael.michaud@orange.fr>
|
This is certainly a very thorough PR. However, I am not sure that An alternative is to implement new Also, |
|
I know JTS is a 2D library by design and z (or m) values will always have a place appart, |
|
My main concern is changing behaviour of I'm not so concerned about Densifier. Although I suspect that Densifier is mostly used to alleviate problems with algorithms which are themselves 2D only, and hence using the Z value might also produce unanticipated results. E.g. one use for Densifier is to allow long line segments to "bend" during reprojection - and reprojection is 2D only, typically. |
Take care of z ordinate in LineSegment. Also benefits Densifier which can now interpolate z values. A hasZ() method has been added in different coordinates implementation : it is not strictly needed but is more readable than cryptic Double.isNaN(d)