Skip to content

Fix broken link in README.md and remove invalid flag from Python example #2396

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

Merged
merged 3 commits into from
Aug 30, 2024

Conversation

UsernamesLame
Copy link
Contributor

@UsernamesLame UsernamesLame commented Aug 29, 2024

Fix broken C-style API link in README.md, and remove nonexistent whisper flags from python example.

Fix broken C-style API link
Update examples/python/whisper_processor.py to remove nonexistent flag "-np" from subprocess.Popen call.
@UsernamesLame UsernamesLame changed the title Update README.md Update README.md and Python example Aug 29, 2024
@UsernamesLame UsernamesLame changed the title Update README.md and Python example Fix broken link in README.md and remove invalid flag from Python example Aug 29, 2024
@UsernamesLame
Copy link
Contributor Author

This PR fixes #2395 and #2394.

abdeladim-s/pywhispercpp wasn't added to the list / was removed at some point (?)

It was referenced in issue ggml-org#9, so I feel like it's worthy of being added as it's the first if not one of the first Python wrappers for whisper.cpp
@ggerganov ggerganov merged commit 9600fc3 into ggml-org:master Aug 30, 2024
bygreencn added a commit to bygreencn/whisper.cpp that referenced this pull request Sep 3, 2024
* ggerganov/master: (40 commits)
  revert : cmake : set MSVC to use UTF-8 on source files (ggml-org#2346)
  sync : ggml
  ggml: fix ggml_graph_cpy undefined behavior (ggml/943)
  cann : fix doxy (ggml/0)
  vulkan : fix build (llama/0)
  cuda : mark BF16 CONT as unsupported
  ggml : fix cont with transposed tensors when one dimension is 1 (ggml/934)
  cmake : set MSVC to use UTF-8 on source files (ggml-org#2346)
  readme : remove invalid flag from Python example (ggml-org#2396)
  readme : fix link (ggml-org#2394)
  go : add beamsize/entropythold/maxcontext to context interface (ggml-org#2350)
  talk-llama : sync llama.cpp
  whisper : update FA call
  sync : ggml
  sync : vulkan (skip) (llama/0)
  ggml : do not crash when quantizing q4_x_x with an imatrix (llama/9192)
  metal : separate scale and mask from QKT in FA kernel (llama/9189)
  ggml : add SSM Metal kernels (llama/8546)
  metal : gemma2 flash attention support (llama/9159)
  CPU/CUDA: Gemma 2 FlashAttention support (llama/8542)
  ...
iThalay pushed a commit to iThalay/whisper.cpp that referenced this pull request Sep 23, 2024
* Update README.md

Fix broken C-style API link

* Update whisper_processor.py

Update examples/python/whisper_processor.py to remove nonexistent flag "-np" from subprocess.Popen call.

* Add pywhispercpp to the Pybind11 Python wrapper list

abdeladim-s/pywhispercpp wasn't added to the list / was removed at some point (?)

It was referenced in issue ggml-org#9, so I feel like it's worthy of being added as it's the first if not one of the first Python wrappers for whisper.cpp
iThalay pushed a commit to iThalay/whisper.cpp that referenced this pull request Sep 23, 2024
* Update README.md

Fix broken C-style API link

* Update whisper_processor.py

Update examples/python/whisper_processor.py to remove nonexistent flag "-np" from subprocess.Popen call.

* Add pywhispercpp to the Pybind11 Python wrapper list

abdeladim-s/pywhispercpp wasn't added to the list / was removed at some point (?)

It was referenced in issue ggml-org#9, so I feel like it's worthy of being added as it's the first if not one of the first Python wrappers for whisper.cpp
lyapple2008 pushed a commit to lyapple2008/whisper.cpp.mars that referenced this pull request Nov 2, 2024
* Update README.md

Fix broken C-style API link

* Update whisper_processor.py

Update examples/python/whisper_processor.py to remove nonexistent flag "-np" from subprocess.Popen call.

* Add pywhispercpp to the Pybind11 Python wrapper list

abdeladim-s/pywhispercpp wasn't added to the list / was removed at some point (?)

It was referenced in issue ggml-org#9, so I feel like it's worthy of being added as it's the first if not one of the first Python wrappers for whisper.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants