Skip to content
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

Added local_files_only option on whisperx.load_model for offline mode #867

Merged
merged 2 commits into from
Jan 1, 2025

Conversation

RoqueGio
Copy link
Contributor

Adds the parameter local_files_only (default False for consistency) to whisperx.load_model so that the user can avoid downloading the file and return the path to the local cached file if it exists.

…o whisperx.load_model so that the user can avoid downloading the file and return the path to the local cached file if it exists.
madriss

This comment was marked as resolved.

Copy link
Collaborator

@Barabazs Barabazs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one minor change request please.

whisperx/asr.py Outdated Show resolved Hide resolved
@Barabazs Barabazs merged commit 0fdb55d into m-bain:main Jan 1, 2025
@Barabazs
Copy link
Collaborator

Barabazs commented Jan 1, 2025

Thank you for your contribution @RoqueGio !

yuki-opus pushed a commit to opus-pro/whisperX that referenced this pull request Jan 2, 2025
…mode (m-bain#867)

Adds the parameter local_files_only (default False for consistency) to whisperx.load_model so that the user can avoid downloading the file and return the path to the local cached file if it exists.

---------

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>
yuki-opus added a commit to opus-pro/whisperX that referenced this pull request Jan 6, 2025
* local vad model

* move model to assets

* Remove typo in error message

* Fix link in README.md

* Added Romanian phoneme-based ASR model (m-bain#791)

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: add new align models (m-bain#922)

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: update Norwegian models (m-bain#687)

Updated Norwegian Bokmål and Norwegian Nynorsk models

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* fix: Force ctranslate to version 4.4.0

Force ctranslate to version 4.4.0 due libcudnn_ops_infer.so.8:
SYSTRAN/faster-whisper#729

Co-authored-by: Icaro Bombonato <ibombonatosites@gmail.com>

* Update MANIFEST.in to include necessary files

* chore: bump version

* feat: update faster-whisper to 1.0.2 (m-bain#814)

* Update faster-whisper to 1.0.2 to enable model distil-large-v3

* feat: add hotwords option to default_asr_options

---------

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: add support for faster-whisper 1.0.3 (m-bain#875)




---------

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: update versions for pyannote:3.3.2 and faster-whisper:1.1.0 (m-bain#936)

* chore: bump faster-whisper to 1.1.0

* chore: bump pyannote to 3.3.2

* feat: add multilingual option in load_model function

---------

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: add verbose output (m-bain#759)




---------

Co-authored-by: Abhishek Sharma <abhishek@zipteams.com>
Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: add local_files_only option on whisperx.load_model for offline mode (m-bain#867)

Adds the parameter local_files_only (default False for consistency) to whisperx.load_model so that the user can avoid downloading the file and return the path to the local cached file if it exists.

---------

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>

* feat: use model_dir as cache_dir for wav2vec2 (m-bain#681)

* feat: add Python compatibility testing workflow

feat: restrict Python versions to 3.9 - 3.12

* feat: add build and release workflow

* chore: clean up MANIFEST.in by removing unnecessary asset inclusions

* chore: update gitignore

* fix: update README image source and enhance setup.py for long description

* docs: update installation instructions in README

* chore: update license in setup.py

* fix: add UTF-8 encoding when reading README.md

* chore: update ctranslate2 version to restrict <4.5.0

* chore: bump whisperX to 3.3.0

* fix: update import statement for conjunctions module

* refactor: simplify imports for better type inference

* refactor: add type hints

* feat: include speaker information in  WriteTXT when diarizing

* refactor: replace NamedTuple with TranscriptionOptions in FasterWhisperPipeline

---------

Co-authored-by: Max Bain <max@reka.ai>
Co-authored-by: Max Bain <36994049+m-bain@users.noreply.github.com>
Co-authored-by: Alex Zamoshchin <alexzamoshchin@gmail.com>
Co-authored-by: Jim O’Regan <jaoregan@tcd.ie>
Co-authored-by: Ruhollah Majdoddin <r.majdodin@gmail.com>
Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>
Co-authored-by: Ismael Ruiz Ranz <35577277+Equipo45@users.noreply.github.com>
Co-authored-by: pere <per@capia.no>
Co-authored-by: Icaro Bombonato <ibombonatosites@gmail.com>
Co-authored-by: Frost Ming <mianghong@gmail.com>
Co-authored-by: moritzbrantner <31051084+moritzbrantner@users.noreply.github.com>
Co-authored-by: Hasan Naseer <103447984+Hasan-Naseer@users.noreply.github.com>
Co-authored-by: Abhishek Sharma <34185691+H4CK3Rabhi@users.noreply.github.com>
Co-authored-by: Abhishek Sharma <abhishek@zipteams.com>
Co-authored-by: Roque Giordano <43444235+RoqueGio@users.noreply.github.com>
Co-authored-by: bnitsan <88460625+bnitsan@users.noreply.github.com>
Co-authored-by: Philippe Anel <zexigh@gmail.com>
croquies pushed a commit to fika-dev/x-custom that referenced this pull request Jan 20, 2025
…mode (m-bain#867)

Adds the parameter local_files_only (default False for consistency) to whisperx.load_model so that the user can avoid downloading the file and return the path to the local cached file if it exists.

---------

Co-authored-by: Barabazs <31799121+Barabazs@users.noreply.github.com>
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.

3 participants