-
Notifications
You must be signed in to change notification settings - Fork 558
Add download stats filter for MagentaRT #1620
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
Conversation
pcuenca
left a comment
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.
Looks good! We'd also need a corresponding PR in https://huggingface.co/google/magenta-realtime to set the library name, wouldn't we?
|
Library name has been set, thanks! https://huggingface.co/google/magenta-realtime/commit/c1578ac0682a4849f6313d8e57ff9b6c6139fcc8 |
Wauplin
left a comment
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.
LGTM!
|
@Wauplin thanks for the review! I'm one of the Magenta RealTime owners, looking forward to getting this merged. Do you happen to have any insights on why the browser test is failing on this case? |
|
Hi @KEHANG , failing CI is unrelated so we can merge it now :) Expect 3-4 days before getting it live on the Hub! |
|
Thanks @Wauplin ! |
No description provided.