street-lab-tech / street-whisper-app

STREET Lab Whisper App - a tool that transcribes and translates audio files into CSV files.
MIT License
2 stars 0 forks source link

Enhancement: Providing premature exit option for CLI #10

Closed carmen-chau closed 4 months ago

carmen-chau commented 4 months ago

Look into whether implementing a feature where a user can prematurely exit the CLI prior to reaching the prompt that appears right before transcription/translation appears:

Do you want to start the process? If not, this app will quit.

Currently, if the user wishes to terminate CLI prior to this process, they would have to press CTRL-C 2-3 times to exit (the program also throws errors due to interrupted termination)

Possible implementation ideas:

carmen-chau commented 4 months ago

@moonsdust thoughts about this issue? I think we should run it by the STREET team to see if this is a pressing issue to address, and how we can approach it.

moonsdust commented 4 months ago

@moonsdust thoughts about this issue? I think we should run it by the STREET team to see if this is a pressing issue to address, and how we can approach it

@carmen-chau In the PR I made, I added in different ways the app can exit with throwing errors. But I had it so that for the list-type prompts, there's an option to exit the app and for the input-type prompts, users can just use CTRL-C to exit the app and it shouldn't throw an error (hopefully).