[BE][CI] remove flakiness calculation in print_test_stats.py#79905
Closed
janeyx99 wants to merge 1 commit intopytorch:masterfrom
Closed
[BE][CI] remove flakiness calculation in print_test_stats.py#79905janeyx99 wants to merge 1 commit intopytorch:masterfrom
janeyx99 wants to merge 1 commit intopytorch:masterfrom
Conversation
Contributor
🔗 Helpful links
✅ No Failures (1 Pending)As of commit 29a5f5d (more details on the Dr. CI page): Expand to see more💚 💚 Looks good so far! There are no failures yet. 💚 💚 This comment was automatically generated by Dr. CI (expand for details).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
f2bae7c to
29a5f5d
Compare
Contributor
Author
|
@pytorchbot merge -g |
Collaborator
|
@pytorchbot successfully started a merge job. Check the current status here |
Collaborator
|
@janeyx99 your PR has been successfully merged. |
Contributor
|
Hey @janeyx99. |
facebook-github-bot
pushed a commit
that referenced
this pull request
Jun 22, 2022
…#79905) Summary: We now use Rockset to store test case stats and we can glean flakiness from that data, so this complicated logic isn't necessary anymore! This will also prevent spurious red on trunk that occurs with this code e.g., https://github.com/pytorch/pytorch/runs/6968080598?check_suite_focus=true ## test plan print_test_stats work on the PR: https://github.com/pytorch/pytorch/runs/6974961237?check_suite_focus=true though it seems several jobs ran into HTTP 500 and Memory errors which I don't think is related to my code + more a problem with the server https://github.com/pytorch/pytorch/runs/6974938954?check_suite_focus=true Pull Request resolved: #79905 Approved by: https://github.com/zengk95, https://github.com/seemethere Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/4f08d56183a19518dba65d692f9dd23d7308fbdb Reviewed By: atalman Differential Revision: D37327501 Pulled By: janeyx99 fbshipit-source-id: c55e4d027bd1a2ed3f840a51f31f78c94b014fac
miladm
pushed a commit
to miladm/pytorch
that referenced
this pull request
Jun 27, 2022
…#79905) We now use Rockset to store test case stats and we can glean flakiness from that data, so this complicated logic isn't necessary anymore! This will also prevent spurious red on trunk that occurs with this code e.g., https://github.com/pytorch/pytorch/runs/6968080598?check_suite_focus=true ## test plan print_test_stats work on the PR: https://github.com/pytorch/pytorch/runs/6974961237?check_suite_focus=true though it seems several jobs ran into HTTP 500 and Memory errors which I don't think is related to my code + more a problem with the server https://github.com/pytorch/pytorch/runs/6974938954?check_suite_focus=true Pull Request resolved: pytorch#79905 Approved by: https://github.com/zengk95, https://github.com/seemethere
laurentdupin
pushed a commit
to laurentdupin/pytorch
that referenced
this pull request
Apr 25, 2026
…#79905) We now use Rockset to store test case stats and we can glean flakiness from that data, so this complicated logic isn't necessary anymore! This will also prevent spurious red on trunk that occurs with this code e.g., https://github.com/pytorch/pytorch/runs/6968080598?check_suite_focus=true ## test plan print_test_stats work on the PR: https://github.com/pytorch/pytorch/runs/6974961237?check_suite_focus=true though it seems several jobs ran into HTTP 500 and Memory errors which I don't think is related to my code + more a problem with the server https://github.com/pytorch/pytorch/runs/6974938954?check_suite_focus=true Pull Request resolved: pytorch#79905 Approved by: https://github.com/zengk95, https://github.com/seemethere
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
5 participants
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.
We now use Rockset to store test case stats and we can glean flakiness from that data, so this complicated logic isn't necessary anymore!
This will also prevent spurious red on trunk that occurs with this code e.g., https://github.com/pytorch/pytorch/runs/6968080598?check_suite_focus=true
test plan
print_test_stats work on the PR: https://github.com/pytorch/pytorch/runs/6974961237?check_suite_focus=true though it seems several jobs ran into HTTP 500 and Memory errors which I don't think is related to my code + more a problem with the server https://github.com/pytorch/pytorch/runs/6974938954?check_suite_focus=true