[JAX] Use "nyu-mll/glue" instead of "glue" for encoder datasets to fix 404 error#2625
Merged
jberchtold-nvidia merged 3 commits intoNVIDIA:mainfrom Jan 27, 2026
Conversation
Contributor
Greptile OverviewGreptile SummaryThis PR updates HuggingFace dataset paths to use the new organization-prefixed format (
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Test as Test Script
participant HF as HuggingFace Datasets API
participant Dataset as Dataset Repository
Note over Test,Dataset: Before: Using deprecated paths
Test->>HF: load_dataset("glue", "cola")
HF->>Dataset: Request from old path
Dataset-->>HF: 404 Not Found
HF-->>Test: Error
Note over Test,Dataset: After: Using new organization paths
Test->>HF: load_dataset("nyu-mll/glue", "cola")
HF->>Dataset: Request from nyu-mll/glue
Dataset-->>HF: Return dataset
HF-->>Test: Dataset loaded
Test->>HF: load_dataset("ylecun/mnist")
HF->>Dataset: Request from ylecun/mnist
Dataset-->>HF: Return dataset
HF-->>Test: Dataset loaded
|
…error Signed-off-by: Jeremy Berchtold <[email protected]>
bc3729d to
4c8eb15
Compare
Collaborator
Author
|
/te-ci L0 jax |
Signed-off-by: Jeremy Berchtold <[email protected]>
Collaborator
Author
|
/te-ci L0 jax |
Signed-off-by: Jeremy Berchtold <[email protected]>
| @@ -0,0 +1,3 @@ | |||
| # Datasets used by TE encoder tests. Pull these to pre-emptively cache datasets | |||
| ylecun/mnist | |||
| nyu-mll/glue No newline at end of file | |||
Contributor
There was a problem hiding this comment.
missing newline at end of file
Suggested change
| nyu-mll/glue | |
| nyu-mll/glue | |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Collaborator
Author
|
/te-ci L0 jax |
Collaborator
KshitijLakhani
left a comment
There was a problem hiding this comment.
LGTM !
Thanks Jeremy
KshitijLakhani
approved these changes
Jan 27, 2026
6 tasks
KshitijLakhani
pushed a commit
that referenced
this pull request
Jan 28, 2026
…x 404 error (#2625) * Use "nyu-mll/glue" instead of "glue" for encoder datasets to fix 404 error Signed-off-by: Jeremy Berchtold <[email protected]> * rename mnist dataset path Signed-off-by: Jeremy Berchtold <[email protected]> * add dataset manifest Signed-off-by: Jeremy Berchtold <[email protected]> --------- Signed-off-by: Jeremy Berchtold <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
HF "glue" dataset seems to have moved from "glue" to "nyu-mll/glue". Small PR to update this dataset path as we've started to see 404 errors
Type of change
Changes
Checklist: