Repository navigation
Studio: show the AMD NPU by default again, with a clearer setup notice - #14524
danielhanchen wants to merge 7 commits into
Conversation
UNSLOTH_STUDIO_ENABLE_NPU=0 still hides it.
|
The account paying for this security review has reached its Codex usage limits. The payer can check the Codex usage dashboard. For personal accounts, using credits requires enabling “Use credits for security reviews” in Code review settings. If you do not manage the paying account, contact this repository's admins. |
|
@codex review |
1 similar comment
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 81b3e72e55
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| {setUp ? ( | ||
| <p className="text-muted-foreground"> | ||
| These models run on the AMD NPU, separately from the GPU models below. | ||
| </p> |
There was a problem hiding this comment.
Render the ready-state notice in the Hub
The new ready-state copy can never appear in the Hub: NpuCatalogList.body() renders NpuSetupNotice only while catalog.ready is false, whereas this branch requires ready && status.runtime_installed. After setup succeeds, the Hub jumps directly to its spinner or model rows and omits the new NPU/GPU separation explanation; include the notice on the Hub's ready path as well.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Main renders no NPU notice in the Hub once the runtime is ready either, so this is not a change from this PR; leaving the Hub's ready state as it is.
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
1 similar comment
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
1 similar comment
|
@codex review |

Since 2026.10.7 the NPU section is gone from the model picker and the Hub on every Ryzen AI machine, including installs that were already chatting on the NPU. #13797 put it behind
UNSLOTH_STUDIO_ENABLE_NPU=1, so/api/npu/statusreportssupported: falseon an XDNA2 NPU unless that variable is set, and nothing in the app says so.This brings the NPU back by default and makes the first-run notice clearer.
Changes
Backend and CLI
UNSLOTH_STUDIO_ENABLE_NPU=0(orfalse,no,off) still hides it, and NPU loads, the runtime start andunsloth run/unsloth chatwith an NPU id are refused with a message naming the variable.Setup notice (model picker NPU section and the Hub NPU list)
Testing
studio/backend/tests/test_npu_detect.py,test_npu_backend.py,test_npu_chat_route.py,test_flm_files.pyandunsloth_cli/tests/test_npu_cli.pypass. The detection test now covers unset, empty,1,0,falseandOff; on main the unset and empty cases fail, since main hides the NPU there.test_export_npu_q4nx.py::test_missing_converter_deps_fail_before_installingfails on main too and is unrelated.Screenshots
Two isolated installs, main (
ab26545) and this branch. This build host has no NPU, so both Studios were started with the backend's Linux PCI probe reporting a Strix Halo XDNA2 NPU; the gate,/api/npu/statusand the frontend are each side's own code. Nothing was enabled or downloaded.Model picker, Recommended tab, NPU section opened, with
UNSLOTH_STUDIO_ENABLE_NPUunset. Main shows no NPU section at all; this branch shows it with the new card:The same with
UNSLOTH_STUDIO_ENABLE_NPU=1on both sides, so main's notice is visible too. The runtime names are gone, and the card says what the NPU is for and what Enable does: