frozenleaves/pytorch
1
0
Fork 0
mirror of https://github.com/pytorch/pytorch.git synced 2025-10-20 21:14:14 +08:00
Code Issues Packages Projects Releases Wiki Activity

[ez] Remove print in heuristics aggregation (#110621)

Browse Source 
Move print to the beginning instead because putting it at the end makes it so you have to scroll through when debugging, and nothing in that function indicates that it should be printing anything

Also the line for printing disabled issues out of the for loop
Pull Request resolved: https://github.com/pytorch/pytorch/pull/110621
Approved by: https://github.com/huydhn
This commit is contained in:
Catherine Lee
2023-10-06 02:04:49 +00:00
committed by PyTorch MergeBot
parent dac895c10a
commit 8a09fe4a05
3 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tools/stats/import_test_stats.py
View File

@ -86,9 +86,10 @@ def get_disabled_tests(
) -> Optional[Dict[str, Any]]:
def process_disabled_test(the_response: Dict[str, Any]) -> Dict[str, Any]:
# remove re-enabled tests and condense even further by getting rid of pr_num
disabled_issues = get_disabled_issues()
disabled_test_from_issues = dict()
for test_name, (pr_num, link, platforms) in the_response.items():
if pr_num not in get_disabled_issues():
if pr_num not in disabled_issues:
disabled_test_from_issues[test_name] = (
link,
platforms,