You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2022/03/31 17:38:13 UTC

[GitHub] [incubator-sedona] umartin opened a new pull request #596: [SEDONA-96] Refactor ST_MakeValid to use GeometryFixer.

umartin opened a new pull request #596:
URL: https://github.com/apache/incubator-sedona/pull/596


   I will always return a single geometry and works for all geometry types.
   
   ## Is this PR related to a proposed Issue?
   
   SEDONA-96
   
   ## What changes were proposed in this PR?
   
   Refactor ST_MakeValid to use GeometryFixer in JTS
   
   ## How was this patch tested?
   
   Unit tests added
   
   ## Did this PR include necessary documentation updates?
   
   Yes
   


-- 
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: dev-unsubscribe@sedona.apache.org

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

[GitHub] [incubator-sedona] jiayuasu commented on pull request #596: [SEDONA-96] Refactor ST_MakeValid to use GeometryFixer.

Posted by GitBox <gi...@apache.org>.

jiayuasu commented on pull request #596:
URL: https://github.com/apache/incubator-sedona/pull/596#issuecomment-1086695101


   @netanel246 The current MakeValid function was written by yo (with a generator). Do you have any suggestion with this new MakeValid function?


-- 
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: dev-unsubscribe@sedona.apache.org

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