petern48 commented on PR #183: URL: https://github.com/apache/sedona-db/pull/183#issuecomment-3369595893
Looks great. I just remembered, we should also add a new `test_st_azimuth` test in [test_functions.py](https://github.com/apache/sedona-db/blob/7f911353335bd2fb51148aadb39a858a5ec12ffd/python/sedonadb/tests/functions/test_functions.py#L120). We use this file for comparing the outputs directly with PostGIS results. There's also already a `numeric_epsilon` parameter that you can use to relax the precision (see `test_st_buffer` for an example). We often have a more comprehensive set of tests there since adding test cases is much more concise in Python. You can follow these directions in the [contributors-guide.md](https://github.com/apache/sedona-db/blob/7f911353335bd2fb51148aadb39a858a5ec12ffd/docs/contributors-guide.md#python) for testing Python. It will also require a running instance of PostGIS, which you can spin up using Docker by following these [instructions](https://github.com/apache/sedona-db/pull/184) I'm adding. -- 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]
