Commit a9ec424
authored
[EZ] Fix TorchInductor CH queries for A100 (#6938)
As reported by @aorenste where [a100 results were not loaded on the
dashboard](https://hud.pytorch.org/benchmark/compilers?dashboard=torchinductor&startTime=Wed%2C%2009%20Jul%202025%2023%3A22%3A18%20GMT&stopTime=Wed%2C%2016%20Jul%202025%2023%3A22%3A18%20GMT&granularity=hour&mode=inference&dtype=bfloat16&deviceName=cuda%20(a100)&lBranch=main&lCommit=&rBranch=main&rCommit=).
The bug here is in the SQL queries where `arch` is now set correctly to
`a100` by HUD. On the other hand, this legacy field in the database is
empty because `a100` was the only arch in the past and wasn't set
explicitly.
With the work that @yangw-dev plans to do, these queries would likely
not staying around for much longer. So, I just opt for the easy fix of
handling `a100` explicitly.
### Testing
[a100](https://torchci-git-fork-huydhn-fix-a100-query-fbopensource.vercel.app/benchmark/compilers?dashboard=torchinductor&startTime=Wed%2C%2009%20Jul%202025%2023%3A59%3A25%20GMT&stopTime=Wed%2C%2016%20Jul%202025%2023%3A59%3A25%20GMT&granularity=hour&mode=inference&dtype=bfloat16&deviceName=cuda%20(a100)&lBranch=main&lCommit=&rBranch=main&rCommit=)
shows up correctly now
cc @aorenste
Signed-off-by: Huy Do <huydhn@gmail.com>1 parent 456e399 commit a9ec424
2 files changed
Lines changed: 2 additions & 2 deletions
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
76 | 76 | | |
77 | 77 | | |
78 | 78 | | |
79 | | - | |
| 79 | + | |
80 | 80 | | |
81 | 81 | | |
82 | 82 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
17 | | - | |
| 17 | + | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| |||
0 commit comments