-
Notifications
You must be signed in to change notification settings - Fork 61
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
update fairseq to stable commit #135
base: main
Are you sure you want to change the base?
Conversation
@osanseviero I wonder if there is any way to trigger automatic test in action from this fork? |
@bayartsogt-ya , the test is run. It's the red one. The current problem is that the docker takes too long to spawn for the test to pass (it's because the docker downloads a lot of files even on subsequent reloads). If you want to test manually if it works for model
Does that help ? |
Here is a log of fairseq-docker build action:
I cannot see |
I have no idea. Maybe git blame can help ? It's possible it's an old relic of code when fairseq hub support was on a specific branch ? |
looks like changes were recent ones. |
cc: @cndn |
Issue #134
Current version of
fairseq
is causing issues in STT pipeline so I changed it to the last commit of stable release offairseq
.current:
d47119871c2ac9a0a0aa2904dd8cfc1929b113d9
last:
59d966a92aabc68b6e0fe1f7bc3eeccbbbe91413
Error reproduce:
https://www.kaggle.com/code/bayartsogtya/notebooke77335cbe2/notebook
version 1 -> d471198
version 2 -> 59d966
link to huggingface failed example