-
Notifications
You must be signed in to change notification settings - Fork 614
[JAX] Use "nyu-mll/glue" instead of "glue" for encoder datasets to fix 404 error #2625
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
base: main
Are you sure you want to change the base?
[JAX] Use "nyu-mll/glue" instead of "glue" for encoder datasets to fix 404 error #2625
Conversation
Greptile OverviewGreptile SummaryThis PR updates HuggingFace dataset paths in JAX examples to fix 404 errors. The GLUE dataset moved from
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Test as Test Script
participant HF as HuggingFace Hub
participant Dataset as Dataset Loader
participant Model as JAX Model
Note over Test,HF: Before Fix (404 Error)
Test->>Dataset: load_dataset("glue", "cola")
Dataset->>HF: Request "glue" dataset
HF-->>Dataset: 404 Not Found
Dataset-->>Test: Error
Note over Test,HF: After Fix (Success)
Test->>Dataset: load_dataset("nyu-mll/glue", "cola")
Dataset->>HF: Request "nyu-mll/glue" dataset
HF-->>Dataset: Dataset data
Dataset-->>Test: Loaded dataset
Test->>Model: Train/evaluate with dataset
Model-->>Test: Results
|
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.
No files reviewed, no comments
…error Signed-off-by: Jeremy Berchtold <jberchtold@nvidia.com>
bc3729d to
4c8eb15
Compare
|
/te-ci L0 jax |
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.
No files reviewed, no comments
timmoon10
left a comment
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.
LGTM
Signed-off-by: Jeremy Berchtold <jberchtold@nvidia.com>
|
/te-ci L0 jax |
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.
No files reviewed, no comments
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: