New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
segmentation fault when trying to fetch dataset #17
Comments
My fault, I have just downloaded the Python API via pip, yet. The command line interpreter will probably work only for a small subset of commands. |
@mika-data: Did you try to build the Python API yourself on your Debian machine? I normally build on Ubuntu, but I would think the differences are minor. |
No, I have downloaded the Python API as a whl as recommended in your installation documentation. I had previously only python3.9 on my machine, after I had build python3.6 and then build sling from source, everything seems to work for me.
|
Hmm, maybe I should test this on Python 3.9. My Ubuntu only has 3.8. |
I have tested it on another debian machine. There it worked fine:
|
So it seems to work on Python 3.8, but fail on Python 3.9, right? |
Yes, I can confirm the bug on a second machine:
|
Let me try to see if I can reproduce this on one of my own machines. |
I can now reproduce the crash. It seems to have something to do with Python type registration in the pysling C extension when running in Python 3.9. |
I seems like you need to build pysling.so using python3.9-dev for it to work with Python 3.9, so I have added support for building pysling.so for Python 3.9. You change |
When I compile from source against Python 3.10 and don't dockerize (pip/venv/...) you might get:
Sadly I don't know much C++, but isn't this just a point of permitting the type conversion? |
Are you using the newest version of the code? Line 130 of pybase.h does not match your error message. Are there any reason that you cannot use the pre-built wheel? |
I've changed pybase to:
To see the error ;) |
I'm sorry, the current sources work perfectly against Python 3.10! PS: Installed via |
@meerfrau: I use wheels instead of setuptools, so you can install SLING with the following command:
I have updated the code to support Python 3.10 by changing |
The text was updated successfully, but these errors were encountered: