-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Python bindings build fix for aarch64 (and others) #3663
base: master
Are you sure you want to change the base?
Conversation
We only support building those arch via cross-compilation. Your fix is for the binding only? |
The fix is for the binding only, yes. But I could successfully compile deepspeech on a Rasp Pi 4 running Ubuntu 20.04 in about 3 hours without problems. It seems to be supported. Do you need the command line how to built it? I also built the tflite version. I also see no other way of doing it because no Python wheels are supported for aarch64. |
We provide them on the github release page, I don't get your point. |
Looking at the release page I cannot find supported wheels for Python 3.8 and Python 3.9. Can you please provide the link? |
We only support Python as the one provided by Raspbian / ARMBian, so 3.7. |
Ok, with my fix then the full built chain would be supported for Python 3.8+ for Ubuntu based devices. I do not think that only supporting Raspbian / ARMBian with Python 3.7 is matching the majority of users for the Raspberry Pi 4. |
And yet that's their distro. The problem is that your fix will make it work but we have no test coverage for building on this hardware, since we only support cross-compilation. |
Well, then I will document the patch at least here or somewhere else if embedded arm devices on recent Python versions are not in the current scope of the Deepspeech project so that other programmers can still use it. If it will not be merged, this can be closed. |
If you are willing to support other versions, you should try and add testing for it with cross-compilation being supported ; we did the move to GitHub Actions explicitely to ease contributions. |
Mozilla, it sounds like communicating around this has been difficult. It would be great if it were supported to build on arm machines. Many people run systems that are aarch64 or armv7l. I see the importance of adding tests. Could any familiar with the testing system give a quick outline of what is needed to add tests for building on arm? Has anybody started this work? |
It is supported: cross-compilation
Please look at the GitHub Actions folder and #3317 |
I tried to compile deepspeech on a Raspberry Pi running Ubuntu 20.04. The created wheel after building the Python bindings used an incorrect naming:
manylinux1_x86_64.whl
although it should belinux_aarch64.whl