linto-ai / whisper-timestamped

Multilingual Automatic Speech Recognition with word-level timestamps and confidence
GNU Affero General Public License v3.0
1.87k stars 149 forks source link

Add automatic publish to PyPI #143

Closed beveradb closed 9 months ago

beveradb commented 10 months ago

Thanks for your work on this project @Jeronymous!

Just as a heads up, there's an existing PyPI project for whisper-timestamped, set up and published to about 5 months ago by @boseguera: https://pypi.org/project/whisper-timestamped/

While I appreciate that project wasn't set up by you, I've personally found it very useful for pulling in whisper-timestamped as a dependency to my other projects (e.g. https://github.com/karaokenerds/python-lyrics-transcriber).

I've now gained joint owner privileges on that PyPI project (thanks Benny!), and I'd love to set up this repo to auto-publish builds to PyPI whenever new commits are merged to master, so folks like me who may have pulled in the dependency using pip install whisper-timestamped can get the latest code more easily 😄

The github actions package this PR proposes just needs a PyPI token (which I can generate and provide you with separately using my own account, or I can invite you as an owner of the PyPI project to use your own token), configured as env var PYPI_API_TOKEN in the github repo settings: https://github.com/marketplace/actions/python-auto-release-pypi-github

What do you think? 🙏

beveradb commented 10 months ago

FYI for now I've just published an updated package to PyPI: https://pypi.org/project/whisper-timestamped/#history

bennyo27 commented 10 months ago

Hello, this is boseguera, I also wanted to mention that I tried reaching out to linto-ai about me setting up this pypi package but never got an email back.

Jeronymous commented 9 months ago

Thanks a lot @beveradb Excellent. Yes let's ship things so that they can be installed easily with pip :)

You can provide the token to me by email (which you should find in the git commits of this repo). I guess I will also have to add the information in the README.

And sorry @bennyo27 I am investigating who is in charge of treating mails sent to hello at linto.ai and why you got no answer. Somebody should come back to you.

bennyo27 commented 9 months ago

Thanks a lot @beveradb Excellent. Yes let's ship things so that they can be installed easily with pip :)

You can provide the token to me by email (which you should find in the git commits of this repo). I guess I will also have to add the information in the README.

And sorry @bennyo27 I am investigating who is in charge of treating mails sent to hello at linto.ai and why you got no answer. Somebody should come back to you.

Oh thank you so much!

Jeronymous commented 9 months ago

@beveradb Any news on that? I haven't received any email with a token.

You can use the email address hello_at_linto.ai for that.

bennyo27 commented 9 months ago

@beveradb Any news on that? I haven't received any email with a token.

You can use the email address hello_at_linto.ai for that.

I can send you the email with the token, perhaps @beveradb is on break? What is the exact email hello@linto.ai?

beveradb commented 9 months ago

Sorry for the slow reply, just had a busy weekend!

Just emailed you (on your gmail from commits) with the token and instructions to decode and set as the PYPI_API_TOKEN github actions repository secret.

Thanks again!

Jeronymous commented 9 months ago

Thanks a lot @beveradb and @bennyo27 :)

Now I guess I have to write a project description somewhere to fill a bit that page: https://pypi.org/project/whisper-timestamped/

bennyo27 commented 9 months ago

No problem! Yes you can send me what you want to add to that page or I can make you an owner of the pypi page also.

On Thu, Nov 30, 2023 at 2:37 PM Jérôme Louradour @.***> wrote:

Thanks a lot @beveradb https://github.com/beveradb and @bennyo27 https://github.com/bennyo27 :)

Now I guess I have to write a project description somewhere to fill a bit that page: https://pypi.org/project/whisper-timestamped/

— Reply to this email directly, view it on GitHub https://github.com/linto-ai/whisper-timestamped/pull/143#issuecomment-1834670919, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJNN24YCXT6GXWMWXWK5J5LYHEDELAVCNFSM6AAAAAA7T4FD56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZUGY3TAOJRHE . You are receiving this because you were mentioned.Message ID: @.***>

Jeronymous commented 9 months ago

Better if I can be owner of the pypi page :) (as I will probably want to modify things by iterations, depending on how things look and how the library evolves)

bennyo27 commented 9 months ago

Of course I just need your pypi username.

On Fri, Dec 1, 2023, 5:06 AM Jérôme Louradour @.***> wrote:

Better if I can be owner of the pypi page :) (as I will probably want to modify things by iterations, depending on how things look and how the library evolves)

— Reply to this email directly, view it on GitHub https://github.com/linto-ai/whisper-timestamped/pull/143#issuecomment-1836089498, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJNN244V6KXT5YOQ2JJLMBDYHHI35AVCNFSM6AAAAAA7T4FD56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWGA4DSNBZHA . You are receiving this because you were mentioned.Message ID: @.***>

Jeronymous commented 9 months ago

OK! I will have to retry later :/ "PyPI new user registration temporarily suspended"