search

bitcointranscripts / tstbtc

This cli app transcribe audio and videos for submission to the bitcointranscripts repo
MIT License
6 stars 8 forks source link

Transcribe fails with "no attribute queue" #124

Closed carlaKC closed 3 months ago

carlaKC commented 3 months ago

Ran into an issue on: 04c3a82 (not present in f905534, so recently introduced)

(venv) ➜  tstbtc git:(main) tstbtc transcribe "PATH" --deepgram --diarize  --loc lightning-specification  -t 'Lightning Specification Meeting' -d  '2024-05-20' -T 'lightning' --noqueue
/Users/carla/Work/bitcointranscripts/tstbtc/venv/lib/python3.11/site-packages/whisper/timing.py:57: NumbaDeprecationWarning: The 'nopython' keyword argument was not supplied to the 'numba.jit' decorator. The implicit default value for this argument is currently False, but it will be changed to True in Numba 0.59.0. See https://numba.readthedocs.io/en/stable/reference/deprecation.html#deprecation-of-object-mode-fall-back-behaviour-when-using-jit for details.
  @numba.jit
Traceback (most recent call last):
  File "/Users/carla/Work/bitcointranscripts/tstbtc/venv/bin/tstbtc", line 5, in <module>
    from transcriber import cli
  File "/Users/carla/Work/bitcointranscripts/tstbtc/venv/lib/python3.11/site-packages/transcriber.py", line 463, in <module>
    cli.add_command(commands.queue)
                    ^^^^^^^^^^^^^^
AttributeError: module 'app.commands' has no attribute 'queue'
kouloumos commented 3 months ago

I sometimes forget that there's another person out there using this tool, so it's great to hear from you. 😄

I had already addressed this issue in a feature branch (see this commit), but it’s going to take a bit of time before it merges with the main branch. In the meantime, I’ve cherry-picked the fix directly to the main branch in this commit, so the problem should be resolved now!

kouloumos commented 3 months ago

Additional productivity tip: tstbtc now supports a configuration file, so by creating the following config.ini you will not need to repeat those flags everytime:

[DEFAULT]
deepgram = True
diarize = True
noqueue = True
carlaKC commented 3 months ago

I sometimes forget that there's another person out there using this tool, so it's great to hear from you. 😄

Greetings from the abyss! And thank you for the quick fix 🫶 Confirmed fixed on 1845a6e.

tstbtc now supports a configuration file, so by creating the following config.ini you will not need to repeat those flags everytime:

Verrynice 👌