Copilot commented on code in PR #2219:
URL: https://github.com/apache/sedona/pull/2219#discussion_r2250404685
##########
python/sedona/geopandas/geoseries.py:
##########
@@ -1382,7 +1382,7 @@ def _row_wise_operation(
Alternatively, we could create 'other' using the same index as
self,
but that would require index=self.index.to_pandas() which is less
scalable.
- default_val : str or None (default "FALSE")
+ default_val : Any (default None)
The value to use if either L or R is null. If None, nulls are not
handled.
Review Comment:
The docstring update changes the type from 'str or None' to 'Any' but the
description still mentions 'The value to use if either L or R is null'. This
should be updated to reflect that any type is now accepted, not just strings.
```suggestion
The value to use if either L or R is null. The value can be of
any type. If None, nulls are not handled.
```
--
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]
