-
Notifications
You must be signed in to change notification settings - Fork 143
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
add no_checks for sub dataset in lvwerra/stack-exchange-paired since … #1003
Conversation
…it does not contain test split Signed-off-by: Wang, Yi A <yi.a.wang@intel.com>
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. |
occurred after huggingface/datasets#6714 is merged. |
same fix in trl huggingface/trl#1670 |
@sywangyi Could you try with the latest release of Datasets (v2.19.2) please? It should solve this issue, i.e. no need for |
yes. works in 2.19.2 |
Great! Could you remove |
ok, I will do it |
Signed-off-by: Wang, Yi A <yi.a.wang@intel.com>
huggingface#1003) Signed-off-by: Wang, Yi A <yi.a.wang@intel.com>
…it does not contain test split
What does this PR do?
Fixes # (issue)
Before submitting