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

Feature/segement printing #876

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Hasan-Naseer
Copy link
Contributor

@Hasan-Naseer Hasan-Naseer commented Sep 6, 2024

  • Added duration printing. set prnt_duration = True in select functions. load_model, load_audio, pad_trim, transcribe
  • Added segment-level printing. Set prnt_segments = True in .transcribe() function for verbose printing

@newtondotcom
Copy link

Looking forward to see this one merged !

@Hasan-Naseer
Copy link
Contributor Author

Looking forward to see this one merged !

You can use the fork https://github.com/Hasan-Naseer/whisperX.git in the meantime! The main branch has the latest features.

@newtondotcom
Copy link

That is true, but the fact is I would love to have a Python library that I can easily use in Docker image. Can you publish yours or would you give me the autorisation to publish yours ?

@Hasan-Naseer
Copy link
Contributor Author

That is true, but the fact is I would love to have a Python library that I can easily use in Docker image. Can you publish yours or would you give me the autorisation to publish yours ?

As in a pypi package? Shouldn't hurt ig. Also, just a suggestion but you can do

pip install git+<url to fork>

as well to get an installable package.

@newtondotcom
Copy link

I forgot this one, no need for pypi package, thank you so much !

@Hasan-Naseer
Copy link
Contributor Author

You're good!

@MahdeenSky
Copy link

Is there a way to enable it in the cli?

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