petern48 commented on PR #2488: URL: https://github.com/apache/sedona/pull/2488#issuecomment-3514971301
Now, let's address the [CI failure](https://github.com/apache/sedona/actions/runs/19247675887/job/55025776659) of your original commit. The reality is that the issue was in the test itself. I could explain why it was causing problems, but honestly it's a little complicated. ```python # 2) Coverage parity — compare only where inputs are valid for both backends nonnull_nonempty_ps = (gs_in.isna() == False) & (gs_in.is_empty == False) ``` Instead, I'd like you take a step back and rewrite the test. Don't overthink it. You honestly really don't need to understand Sedona as a project much to contribute these Geopandas functions. A lot of it is just copy-paste and following patterns. Now I'll ask you to do this: 1) Go to `test_match_geopandas_series.py`, and delete all of the contents of the `test_minimum_bounding_circle` test. 2) Take a look at the following functions in `test_match_geopandas_series.py`. Then, guess how you should implement `test_minimum_bounding_circle`. It's very simple, much simpler than the original test in this PR. You shouldn't need to ask AI at all. You're welcome to copy and paste. https://github.com/apache/sedona/blob/762d6f85004c17b3bafef73dc3bb65a3a058e38d/python/tests/geopandas/test_match_geopandas_series.py#L721-L725 If you're still confused, you're welcome to ask for help. We might need to make some small adjustments to make things pass, but this is a lot closer to what our desired end result is. -- 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]
