Uh oh!
There was an error while loading. Please reload this page.
PYTHON-1359 Fix for numpy>=1.24.0 - #1169
Conversation
weisslj
commented
Jul 9, 2023
The CI fails even after I have reverted my fix commit in 77892a5 for testing, see https://ci.appveyor.com/project/DataStax/python-driver/builds/47482656. |
Minimal `example.py`:
```python
from cassandra.cluster import Cluster
from cassandra.query import tuple_factory
from cassandra.protocol import NumpyProtocolHandler
import pandas as pd
cluster = Cluster()
session = cluster.connect()
session.row_factory = tuple_factory
session.client_protocol_handler = NumpyProtocolHandler
print(pd.DataFrame(session.execute("SELECT broadcast_port FROM system.local").one()))
```
Result with numpy<1.24.0:
```
$ python3 example.py
broadcast_port
0 7000
```
Result with numpy>=1.24.0:
```
$ python3 example.py
/usr/lib/python3/dist-packages/cassandra/io/libevreactor.py:370: FutureWarning: In the future `np.bool` will be defined as the corresponding NumPy scalar. (This may have returned Python scalars in past versions.
self.process_io_buffer()
Empty DataFrame
Columns: []
Index: []
```77892a5 to
39a937eCompareabsurdfarce
commented
Jul 20, 2023
Thanks for the PR @weisslj! Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks! |
@absurdfarce sorry for the long delay, I completely missed the notification! I know have signed the Contributor License Agreement! |
weisslj
commented
Mar 22, 2024
@absurdfarce any chance to get this merged soon? It would help us a lot to have this fix included upstream 🙏 |
absurdfarce
commented
Mar 25, 2024
This is a very nice find indeed @weisslj ! I just looked over the general context of this change and your PR; everything looks good to me. I'm running a test build on our CI infrastructure to confirm that everything behaves as expected and we get the expected improvements in our tests. Assuming there aren't any surprises there (and I'm really not expecting any) I'll merge this straightaway. |
absurdfarce
commented
Mar 26, 2024
Confirmed that the test_cython_protocol_handlers issues are addressed by this patch (as expected). Calling it good and ready to merge this one. Thanks for the fix @weisslj! |
Reported also in https://datastax-oss.atlassian.net/browse/PYTHON-1359, minimal
example.py:Result with numpy<1.24.0:
Result with numpy>=1.24.0: