-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
Reboot Agents #30387
Reboot Agents #30387
Conversation
60ae172
to
5b859e9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall this is starting to look quite good!
Let's make sure there are disclaimers in the docs about loading remote tools, the python interpreter, and anything else that might be a security risk.
I'll write docs about collections as well.
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.


I think I might not have librosa or soundfile.
Return code needs to be supported for easy debug especially here, the errors are not straighforward.

the warning of the validation should be better formatted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can 't we store these on the hub?
d75d85a
to
cabc614
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nits and good to go!
class SpeechToTextTool(Tool): | ||
description = "This is a tool that transcribes an audio into text. It returns the transcribed text." | ||
name = "transcriber" | ||
|
||
inputs = {"audio": {"type": "audio", "description": "The audio to transcribe"}} | ||
output_type = "text" | ||
|
||
def __init__(self, *args, **kwargs): | ||
super().__init__() | ||
self.client = InferenceClient(model="distil-whisper/distil-large-v3") | ||
|
||
def forward(self, audio): | ||
return self.client.automatic_speech_recognition(audio).text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not adressed
Co-authored-by: Arthur <48595927+ArthurZucker@users.noreply.github.com>
Co-authored-by: Arthur <48595927+ArthurZucker@users.noreply.github.com>
Co-authored-by: Arthur <48595927+ArthurZucker@users.noreply.github.com>
Co-authored-by: Arthur <48595927+ArthurZucker@users.noreply.github.com>
Co-authored-by: Arthur <48595927+ArthurZucker@users.noreply.github.com>
You can also check my mini autogpt repo for reevaluating outputs which had low confidence. |
What does this PR do?
Reboot agents in transformers!
cc @LysandreJik