-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
pyarrow: Segmentation fault (core dumped) #1450
Comments
Could we have more information about your platform, and how you installed keras? |
Sure, it's running inside a Docker container. I've pasted the Dockerfile below: Nothing special in the requirements.txt file, just the usual stuff, but I don't pin the versions, I just use the latest via pip :)
|
Here is the requirements.txt file, if it's useful:
|
and the conda_requirements.txt:
|
I would suspect that this issue comes from the tensorflow You should be able to verify this by running the following code. Please do, as if it isn't the above mentioned issue, we might be able to fix it in a simpler fashion.
(Note that instead of For now, ensuring the import order is your easiest way around this. We probably use a C++ symbol in 0.8.0 that we have not been using with 0.7.1. There are two optimal solutions to the this which are rather a long term approach:
|
Should this issue be raised with TF? They claim to be producing manylinux1 wheels but I guess they are not using the standard manylinux1 image setup? |
@wesm Yes but I think that this issue was already raised somewhere. |
I'm looking at https://github.com/tensorflow/tensorflow/tree/master/tensorflow/tools/ci_build It looks like (from what I can see anyway) TensorFlow is not using the same compiler as the manylinux1 spec (devtoolset-2 gcc 4.8.5 I think), which I guess would explain the quagmire we are in. So one possible workaround is that we could try to import tensorflow pre-emptively if it's available when we are importing pyarrow (so that its symbols get loaded first). |
👍 for doing this for Tensorflow as we might get a lot of users that run into this issue. Otherwise the longterm fix is to have |
Since TensorFlow is using a non-standard compiler to make manylinux1 wheels, I'm not sure there's a good solution here except to "import tensorflow before pyarrow" |
I resolved ARROW-1960 as "won't fix" with this resolution for now |
Getting a segmentation fault, when loading this after Keras (see below)
Keras version = 2.1.2
tensorflow version = 1.4.1
pyarrow version = 0.8.0
edit: #1 strangely enough, it seems to run fine if I switch the order of the imports (i.e. importing keras before pyarrow)
edit: #2 it works fine if I drop pyarrow back to '0.7.1'
Please advise
The text was updated successfully, but these errors were encountered: