Skip to content

[Tests] Upload custom test artifacts #572

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

Merged
merged 4 commits into from
Sep 19, 2022
Merged

[Tests] Upload custom test artifacts #572

merged 4 commits into from
Sep 19, 2022

Conversation

anton-l
Copy link
Member

@anton-l anton-l commented Sep 19, 2022

This adds utils to track test failures, durations, etc. Based on the transformers testing utils.

@anton-l anton-l requested a review from patil-suraj September 19, 2022 14:25
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 19, 2022

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

@anton-l anton-l merged commit 0a2c42f into main Sep 19, 2022
tr.summary_warnings() # final warnings

tr.reportchars = "wPpsxXEf" # emulate -rA (used in summary_passes() and short_test_summary())
with open(report_files["passes"], "w") as f:
Copy link
Contributor

Choose a reason for hiding this comment

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

@anton-l Is any particular reason to enable this report_files["passes"]?

Copy link
Contributor

Choose a reason for hiding this comment

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

In transformers, this is disabled. Although enabling it won't affect diffusers for now, I also think this part won't provide further useful information for us.

@kashif kashif deleted the upload-test-artifacts branch September 22, 2022 15:06
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
* make_reports

* add test utils

* style

* style
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