Skip to content

sd: sync to master-355-694f0d9#1841

Closed
wbruna wants to merge 3 commits into
LostRuins:concedo_experimentalfrom
wbruna:kcpp_sd_355-694f0d9
Closed

sd: sync to master-355-694f0d9#1841
wbruna wants to merge 3 commits into
LostRuins:concedo_experimentalfrom
wbruna:kcpp_sd_355-694f0d9

Conversation

@wbruna

@wbruna wbruna commented Nov 10, 2025

Copy link
Copy Markdown

This merge actually didn't result in a lot of conflicts: mostly from added or removed functions near Koboldcpp changes. I also needed to add the new name_conversion.cpp to the Makefile and to sdtype_adapter.cpp.

Didn't test extensively yet, but basic generation seems OK.

As before, updated changes against upstream can be reviewed at:
wbruna/stable-diffusion.cpp@koboldcpp_sd_base...koboldcpp_sd_changes

@LostRuins LostRuins added the enhancement New feature or request label Nov 11, 2025
@wbruna wbruna force-pushed the kcpp_sd_355-694f0d9 branch from 70f2e51 to 220e06f Compare November 11, 2025 11:36
@wbruna

wbruna commented Nov 11, 2025

Copy link
Copy Markdown
Author

Rebased to current concedo_experimental; no new changes.

@wbruna

wbruna commented Nov 11, 2025

Copy link
Copy Markdown
Author

Hold on, LoRAs aren't being applied correctly.

@wbruna wbruna marked this pull request as draft November 11, 2025 12:45
@wbruna wbruna marked this pull request as ready for review November 12, 2025 00:36
@LostRuins

Copy link
Copy Markdown
Owner

just when leejet/stable-diffusion.cpp#969 appeared 😆

im still waiting for my laptop to be fixed, but i'll test this hopefully next week

@wbruna

wbruna commented Nov 12, 2025

Copy link
Copy Markdown
Author

My thoughts exactly 🙂 I can include it in this PR once it's merged.

@wbruna wbruna force-pushed the kcpp_sd_355-694f0d9 branch from 6660fbe to d973bcd Compare November 14, 2025 01:16
@wbruna wbruna closed this Nov 14, 2025
@wbruna wbruna deleted the kcpp_sd_355-694f0d9 branch November 14, 2025 01:19
@wbruna

wbruna commented Nov 14, 2025

Copy link
Copy Markdown
Author

See #1844 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants