ceos/unsloth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge diffusion-sdxl into diffusion-lora-ux; keep options-only LoRA catch

Browse source 
The catalog-refresh .catch from the lower branch clears the selected adapters
too, which is right for its catalog-only picker but wrong here: this picker
holds free-text HF repo ids that are valid without being in the catalog, so a
transient refresh failure must not wipe them. Family swaps still clear the
selection and hidden LoRAs are never sent.
This commit is contained in:
Daniel Han 2026-07-02 01:23:15 +00:00
commit 76b5ee354d
1 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
studio/frontend/src/features/images/images-page.tsx
View file

@ -1015,12 +1015,13 @@ export function ImagesPage({ active = true }: { active?: boolean }) {
if (!cancelled) setAvailableLoras(list);
})
.catch(() => {
if (cancelled) return;
// Clear the SELECTED adapters too, not just the options: leaving a stale `loras`
// selection in state (with the picker now hidden/empty) would still be posted by
// handleGenerate and could apply adapters from the previous model, or fail.
setAvailableLoras([]);
setLoras([]);
// Clear only the OPTIONS on a failed catalog refresh. Unlike the catalog-only
// picker below the stack, this free-text picker holds selections (bare HF repo
// ids) that are valid without being in the catalog; a transient refresh failure
// must not wipe them. Stale cross-family selections are already cleared by the
// family-swap check above, and hidden LoRAs are never sent (handleGenerate is
// gated on loraCapable).
if (!cancelled) setAvailableLoras([]);
});
return () => {
cancelled = true;