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

A script to download artifacts and perform CI error statistics #18865

Merged
merged 1 commit into from
Sep 2, 2022

Conversation

ydshieh
Copy link
Collaborator

@ydshieh ydshieh commented Sep 2, 2022

What does this PR do?

This script is helpful for the past CI project. It downloads all artifacts from a workflow run, and get the error statistics + the corresponding failing tests.

errors.json: the places where error occur + what those errors are

failed_tests.json: which test methods failed (can be used to determine which models will be supported in a specific backend version)

** We might adjust this script a bit once we start to perform the automation of the past CI project. **

Currently, it prints something as the following (but we save the full information in 2 json files)

('RuntimeError: Integer division of tensors using div or / is no longer supported, and in a future release div will perform true division as in Python 3. Use true_divide or floor_divide (// in Python) instead.', 66)
("AttributeError: module 'torch.jit' has no attribute '_state'", 51)
("AttributeError: module 'torch' has no attribute 'minimum'", 45)
("AttributeError: module 'torch' has no attribute 'multiply'", 25)
('AttributeError: Caught AttributeError in replica 0 on device 0.', 3)
('RuntimeError: "normal_kernel_cpu" not implemented for \'BFloat16\'', 3)
("AssertionError: Couldn't trace module.", 3)
("AttributeError: module 'torch' has no attribute 'isneginf'", 2)
("TypeError: where(): argument 'input' (position 2) must be Tensor, not int", 2)
("AttributeError: 'Tensor' object has no attribute 'nansum'", 1)
('RuntimeError: Caught RuntimeError in replica 0 on device 0.', 1)

@ydshieh ydshieh requested a review from LysandreJik September 2, 2022 10:20
required=True,
help="Where to store the downloaded artifacts and other result files.",
)
parser.add_argument(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't like this place much for the security concern. But at this moment, I might be the only person.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 2, 2022

The documentation is not available anymore as the PR was closed or merged.

@ydshieh ydshieh force-pushed the get_ci_error_statistics branch from 8578b9d to c7f6e3f Compare September 2, 2022 12:29
Copy link
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you @ydshieh

@ydshieh ydshieh merged commit 4e29b3f into huggingface:main Sep 2, 2022
@ydshieh ydshieh deleted the get_ci_error_statistics branch September 2, 2022 15:59
oneraghavan pushed a commit to oneraghavan/transformers that referenced this pull request Sep 26, 2022
…ngface#18865)

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
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