Skip to content

refactor: simplify diffusion model runner params#1569

Merged
leejet merged 1 commit into
masterfrom
refactor/diffusion-model-runner-params
May 27, 2026
Merged

refactor: simplify diffusion model runner params#1569
leejet merged 1 commit into
masterfrom
refactor/diffusion-model-runner-params

Conversation

@leejet
Copy link
Copy Markdown
Owner

@leejet leejet commented May 27, 2026

Summary

  • Replace wrapper-only diffusion model classes with DiffusionModelRunner, which inherits GGMLRunner, owns the tensor prefix, and lets concrete runners expose compute(DiffusionParams).
  • Shrink DiffusionParams to common inputs plus typed model-specific extras.
  • Remove ADM channel plumbing from diffusion/conditioner params and use the SDXL ADM width directly where needed.
  • Fix Flex.2 latent concat handling so Flux control-only logic no longer overwrites Flex.2 inpaint/control concat tensors.

Related Issue / Discussion

N/A

Additional Information

N/A

Checklist

@leejet leejet merged commit 55c2aed into master May 27, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant