petern48 commented on PR #476: URL: https://github.com/apache/sedona-db/pull/476#issuecomment-3709125040
Alright, I've added rust tests, refactored the python tests a bit, and did a bit of cleaning up. For the python tests, I went the route of using exact string matching, but removing all white spaces to avoid formatting differences. It may be slightly odd, but I think it's a safe testing approach. Looks like this for example: `LINESTRINGZM(1234,5678)`. After further thought, I don't really want to spend much time trying to fix XYM upstream. I would expect some friction and discussion, and i don't think it's top priority. Not having `XYM` would not be a regression, so I personally think it would be okay to merge this without addressing `XYM`. I filed a ticket (https://github.com/apache/sedona-db/issues/481) to track it. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
