-
Notifications
You must be signed in to change notification settings - Fork 148
CI Tests on Strix Halo #2450
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
base: main
Are you sure you want to change the base?
CI Tests on Strix Halo #2450
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the code was already updated to for strix halo, but it didn't work? (line 152 in the original). There are more lit.cfg.py
to update in programming_guide
and test
.
Yes exactly, I will also update the other |
In the various
lit.cfg.py
, the regex used on the output of thexrt-smi
(orxbutil
) to determine which device we are using is broken for Strix Halo. Even if we don't use Strix Halo for the GitHub action CI, it's useful for developers to run the CI tests locally.Here is the output of
xrt-smi
orxbutil
:Changing
[\|]?(\[.+:.+:.+\]).+\|(RyzenAI-(npu\d)|NPU (\w+))\W*\|
to[\|]?(\[.+:.+:.+\]).+\|(RyzenAI-(npu\d)|NPU ([^\|]+))\W*\|
does the trick to also match Strix Halo (with a.strip()
to remove the trailing white spaces).