Do we still need ci for dataset available on huggingface?
#3620
KennethEnevoldsen
started this conversation in
General
Replies: 3 comments 2 replies
-
|
(@isaac-chung, @Samoed ) |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Yep sounds like a good plan! |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
I think we can add hf token to make it working ref #2950 |
Beta Was this translation helpful? Give feedback.
2 replies
-
|
We have a token so we just need to use it |
Beta Was this translation helpful? Give feedback.
-
|
Yes, I've ctarted pr #3621 |
Beta Was this translation helpful? Give feedback.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Currently, it is almost never correct, and it consistently fails. This was also more of a problem when we didn't self-host the datasets, but now many of the datasets are hosted by us so it would be surprising if it was taken down.
We could replace it with a new test that ensures that all new datasets are on
mtebor other trusted orgs (e.g.vidore). We can of course still make exceptions.Beta Was this translation helpful? Give feedback.
All reactions