pytorch / test-infra

This repository hosts code that supports the testing infrastructure for the PyTorch organization. For example, this repo hosts the logic to track disabled tests and slow tests, as well as our continuation integration jobs HUD/dashboard.
https://hud.pytorch.org/
Other
83 stars 83 forks source link

[HUD] Merge LF and non LF columns #5857

Closed clee2000 closed 1 month ago

clee2000 commented 1 month ago

The logic here in HUD is pretty bad...

Merges LF and non LF columns into one column on the man page of HUD to reduce the size of HUD. Using it is controlled through checkbox like most other settings

This desperately needs a second pair of eyes

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
torchci ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 4, 2024 11:34pm
huydhn commented 1 month ago

I wonder if we should keep the first 3 fields in the job name like pull / linux-focal-cuda12.1-py3.10-gcc9-sm86 / test (default, 2, 5). It should unique identify the job on HUD.

Additional information like runner, owners (from https://github.com/pytorch/pytorch/pull/139519), or special configs (rerun_disable_test, memory_leak_check) can go to a different part of https://github.com/pytorch/test-infra/blob/main/torchci/components/JobLinks.tsx#L13. Something like:

[failure][rerun_disable_test][owner][linux.g5.4xlarge.nvidia.gpu] pull / linux-focal-cuda12.1-py3.10-gcc9-sm86 / test (default, 2, 5) 
Commit | Raw logs | more like this | Duration: 1.9h