- Notifications
You must be signed in to change notification settings - Fork 483
Seamless texture generation support for qwen image, z-image, and flux#914
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Uh oh!
There was an error while loading. Please reload this page.
Conversation
Phylliida commented Oct 24, 2025 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
Phylliida commented Oct 24, 2025
Some conflicts, lemme resolve those... |
leejet commented Nov 16, 2025
Nice Work! Once ggml merges the circular-pad changes, I will merge this PR. |
oscarbg commented Dec 7, 2025 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
llama.cpp PR merged already! |
Phylliida commented Dec 7, 2025
Needs to be modified slightly bc you can do it only circular pad no circular conv |
Phylliida commented Dec 7, 2025 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
In particular like this: I can clean up this PR to that on wednesday |
leejet commented Dec 9, 2025
I think we can add a field in GGMLRunnerContext to control whether circular padding is used. |
Phylliida commented Dec 10, 2025 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
looked into this, though it was merged into llama.cpp, we'll need to wait for ggml to be synced with llama.cpp's ggml (it seems to only be synced every few weeks), or alternatively adjust the reference to point to llama.cpp's ggml folder instead of ggml |
pwilkin commented Dec 11, 2025
@ggerganov any chance for a GGML sync? |
ggerganov commented Dec 11, 2025
It's synced now - thanks |
Phylliida commented Dec 12, 2025
in progress, wait a few min... |
Phylliida commented Dec 13, 2025
Works for z-image too now ![]() (You can use this to test out seamless textures) |
SkutteOleg commented Dec 13, 2025
Is there any way to make it circular only horizontally? To generate equirectangular 360 panoramas, for example? |
Phylliida commented Dec 13, 2025
In principle it would be fairly easy, but does require an additional pr to llama.cpp |
Phylliida commented Dec 13, 2025
Phylliida commented Dec 13, 2025
SkutteOleg commented Dec 13, 2025
Phylliida commented Dec 13, 2025
You can try it now, but I think my implementation for chroma isn't quite right/we're still missing something because it's mostly seamless but still a little blurry around borders |
Phylliida commented Dec 13, 2025
Ok there's some source of non-circular I'm still missing for chroma, it mostly tiles is a little fuzzy at the edges (maybe something with the vae?), but otherwise things are refactored into -circularx or -circulary or -circular for qwen, z-image, and chroma/flux |
SkutteOleg commented Dec 13, 2025
pwilkin commented Dec 13, 2025
This is just in time for me wanting to get a seamless background tile for my blog ;) |
50ff966 into leejet:masterUh oh!
There was an error while loading. Please reload this page.
leejet commented Dec 21, 2025
Thank you for your contribution. |











Adds support for qwen image to generate seamless textures, using a
--circularflag. Edit: Also includes a--circularxand--circularyif you want only tiling on those axesRequires this PR ggml-org/ggml#1374 (Edit: see ggml-org/llama.cpp#16985) for ggml that adds a "circular" mode that can be used.
I also had to tweak rope so the period of the sinusods would evenly tile.