Skip to content
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

run_gpt2_from_huggingface.py: do not download the onnx data file if it exists #2996

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

tungld
Copy link
Collaborator

@tungld tungld commented Nov 1, 2024

This patch updates the python script run_gpt2_from_huggingface.py so that it does not download .onnx_data files if they exist.

Copy link
Collaborator

@AlexandreEichenberger AlexandreEichenberger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlexandreEichenberger AlexandreEichenberger merged commit d25c63e into onnx:main Nov 1, 2024
6 of 7 checks passed
@jenkins-droid
Copy link
Collaborator

Jenkins Linux s390x Build #15937 [push] Do not download the onnx... started at 12:46

@jenkins-droid
Copy link
Collaborator

Jenkins Linux amd64 Build #15934 [push] Do not download the onnx... started at 11:46

@jenkins-droid
Copy link
Collaborator

Jenkins Linux ppc64le Build #14964 [push] Do not download the onnx... started at 13:00

@jenkins-droid
Copy link
Collaborator

Jenkins Linux s390x Build #15937 [push] Do not download the onnx... failed after 1 hr 3 min

@jenkins-droid
Copy link
Collaborator

Jenkins Linux ppc64le Build #14964 [push] Do not download the onnx... failed after 1 hr 10 min

@jenkins-droid
Copy link
Collaborator

Jenkins Linux amd64 Build #15934 [push] Do not download the onnx... passed after 1 hr 24 min

christopherlmunoz pushed a commit to christopherlmunoz/onnx-mlir that referenced this pull request Nov 4, 2024
Signed-off-by: Tung D. Le <[email protected]>
Co-authored-by: Alexandre Eichenberger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants