AIOS icon indicating copy to clipboard operation
AIOS copied to clipboard

[feat] Transcription Agent example!

Open om-raheja opened this issue 1 year ago • 1 comments

Transcription Agent Description: Can listen to the microphone and execute tasks with the output. On request of @mannanshukla Dependencies: RealtimeSTT

Pretty self explanatory. Tested and works, if any issues come up please don't hesitate to @ me.

om-raheja avatar Aug 20 '24 22:08 om-raheja

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aios-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 22, 2024 7:39pm

vercel[bot] avatar Aug 20 '24 22:08 vercel[bot]

Hey Kai, I renamed the class to Transcriber before submitting the PR and I can't find where it says CurrencyConverter, neither locally nor in my diff nor on GitHub. I made some other needed adjustments to ensure code quality this morning, and I double checked all my class names. If there is anything else, please let me know. Sorry for the inconvenience.

om-raheja avatar Aug 21 '24 14:08 om-raheja

The class you used is called CurrencyConverter, just rename it.

mannanshukla avatar Aug 21 '24 15:08 mannanshukla

Balaji, will do the changes for interact.py. The way the parser is setup is that if no args are passed, the defaults you had before are automatically applied. Check the arg parser code.I can't find where the CurrencyConverter class is. Like I said before, its not in my diffs, my local files, or in GitHub. If you can show me where it is, I'll gladly change it.On Aug 21, 2024, at 11:43, BRama10 @.***> wrote: @BRama10 requested changes on this pull request.

In interact.py, have stderr still print to the console (or at worst, to a logfile), so that in the case of an error due to dependencies (which has happened many times before). On second thought, also have stdout go to a logfile as well. In server.py can you set up default arguments, and only in case of an error (such as the one you were getting), do we fallback onto parser args (since it is a hassle to pass args).

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

om-raheja avatar Aug 21 '24 17:08 om-raheja

Balaji, will do the changes for interact.py. The way the parser is setup is that if no args are passed, the defaults you had before are automatically applied. Check the arg parser code.I can't find where the CurrencyConverter class is. Like I said before, its not in my diffs, my local files, or in GitHub. If you can show me where it is, I'll gladly change it.On Aug 21, 2024, at 11:43, BRama10 @.> wrote: @BRama10 requested changes on this pull request. In interact.py, have stderr still print to the console (or at worst, to a logfile), so that in the case of an error due to dependencies (which has happened many times before). On second thought, also have stdout go to a logfile as well. In server.py can you set up default arguments, and only in case of an error (such as the one you were getting), do we fallback onto parser args (since it is a hassle to pass args). —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.>

I don't think CurrencyConverter ever was a problem (for some reason, your 2nd commit has CurrencyConverter, but your 3rd commit renames it to Transcriptor, so I assume you committed halfway through refactoring the class), so you can safely ignore that!

BRama10 avatar Aug 21 '24 19:08 BRama10