-
Notifications
You must be signed in to change notification settings - Fork 56
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
Adds test cases for t5x finetuning+eval with and without TE #359
base: main
Are you sure you want to change the base?
Conversation
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 we hold it off until the current CI is fully fixed?
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.
Shall we revisit this PR once #401 is merged?
.github/container/test-t5x.sh
Outdated
@@ -7,7 +7,7 @@ print_var() { | |||
} | |||
|
|||
usage() { | |||
echo "Test T5X throughput on a fake-data Wikipedia benchmark." | |||
echo "Test T5X throughput on a fake-data Wikipedia benchmark or real squad." |
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.
What is 'real squad'?
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.
By real squad, I mean "Stanford Question Answering Dataset (SQuAD)". I can use this casing to avoid confusion
No description provided.