Skip to content

Conversation

@richiejp
Copy link
Collaborator

  • ⬆️ Update leejet/stable-diffusion.cpp
  • fix(stablediffusion-ggml): Move parameters and start refactor of passing params

Supersedes: #6264

Also I started refactoring how params are passed because we are running out of arguments when calling C functions and populating C structs from Go is a pain unless they are carefully constructed to have an obvious memory layout.

So this adds some setters for VAE params which presently can't be configured. The setters are called from Go on the various param structs. This adds a lot of boiler plate, but I don't see a nice alternative.

mudler and others added 2 commits September 14, 2025 20:22
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ing params

Signed-off-by: Richard Palethorpe <io@richiejp.com>
@netlify
Copy link

netlify bot commented Sep 15, 2025

Deploy Preview for localai ready!

Name Link
🔨 Latest commit f83cd92
🔍 Latest deploy log https://app.netlify.com/projects/localai/deploys/68c8273759d6be0008f05380
😎 Deploy Preview https://deploy-preview-6271--localai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Richard Palethorpe <io@richiejp.com>
Copy link
Owner

@mudler mudler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 danke!

@richiejp
Copy link
Collaborator Author

I just noticed that upstream the default sampler was set to Euler. Also a default option has been added, so we may just want to remove the euler_a from the flux config?

@mudler
Copy link
Owner

mudler commented Sep 15, 2025

I just noticed that upstream the default sampler was set to Euler. Also a default option has been added, so we may just want to remove the euler_a from the flux config?

sounds reasonable yup

@mudler mudler merged commit 10bf108 into mudler:master Sep 15, 2025
35 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.

2 participants