You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sedona.apache.org by "yutannihilation (via GitHub)" <gi...@apache.org> on 2026/02/14 11:56:57 UTC

[PR] Let CrsTransform handle M coordinate [sedona-db]

yutannihilation opened a new pull request, #619:
URL: https://github.com/apache/sedona-db/pull/619

   This is a followup of #606. Before addressing `ST_Force3DM()` and `ST_Force4D()`, I want to confirm if my approach is correct.
   
   This pull request changes `CrsTransform` to have two additional methods to handle M coordinate (accordingly, `transform_coord_3d()` is renamed to `transform_coord_xyz`). Maybe I could create a separate `transform_m()` method, but I thought it's better to do the transformation in a single call.
   
   - `transform_xym()`
   - `transform_xyzm()`
   
   This also requires `transform_and_write_coords()` to extract and fill M coordinate.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(rust/sedona-geometry): Let CrsTransform handle M coordinate [sedona-db]

Posted by "yutannihilation (via GitHub)" <gi...@apache.org>.
yutannihilation commented on PR #619:
URL: https://github.com/apache/sedona-db/pull/619#issuecomment-3902791700

   Thanks!
   
   > I also wonder if we've made the transform slower
   
   This was my main concern. But, I agree with you that we can optimize later.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(rust/sedona-geometry): Let CrsTransform handle M coordinate [sedona-db]

Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot merged PR #619:
URL: https://github.com/apache/sedona-db/pull/619


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org