-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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 #1524
base: master
Are you sure you want to change the base?
Python Bindings #1524
Conversation
If possible, I think a Python binding test should be added in the CI. |
Yup, would be useful to have a CI.
What are the issues on OSX related to? |
I was able to execute example.py file in python bindings folder with a few small tweaks on windows Copy below dlls to current working directory so
Execution log
|
@vivekuppal Thanks for testing on Windows! |
The autogenerated wrapper is just not loading the library:
Trying to rebuild the wrapper with ctypesgen has the following errors:
Not sure what the issue is! Maybe some developer stuff for C++ is not installed correctly? I'm a Python programmer, and can't assist here. But success so far on both Windows and Linux (Ubuntu). |
The sample code in the README does not work. The wrapper builds fine for me, but is inaccessible.
|
FWIW this runs fine for me on MacOS. I get the same errors as @dnhkng when running ctypesgen, but the generated wrapper works fine anyway, and the example runs fine too, with GPU support and everything. |
The Pybind11 binding from the readme also works for me, as long as I update the whisper.cpp submodule. But no GPU support, sadly. |
What's the status of this PR? Have people been using it successfully? |
|
Currently, all the listed Python bindings on the main ReadMe are broken and do not work on the current API.
This PR removes the links to the non-functional Python-binding repositories, and adds a simple binding system that works on Ubuntu x86 with CUDA.
This still needs testing on Windows, and it's not working on OSX yet, but better than what's currently available.