NHacker Next
login
▲I have reimplemented Stable Diffusion 3.5 from scratch in pure PyTorchgithub.com
397 points by yousef_g 18 hours ago | 69 comments
Loading comments...
liuliu 17 hours ago [-]
If you are interested in this: Flux reference implementation is very minimalistic: https://github.com/black-forest-labs/flux/tree/main/src/flux

The minRF project is very easy to start with training small diffusion models with rectified flow: https://github.com/cloneofsimo/minRF

Also, the reference implementation of SD 3.5 is actually minimalistic too: https://github.com/Stability-AI/sd3-ref

doctorpangloss 14 hours ago [-]
Reference implementations are unmaintained and buggy.

For example https://github.com/huggingface/transformers/issues/27961 OpenAI's tokenizer for CLIP is buggy, it's a reference implementation, it isn't the one they used for training, and the problems with it go unsolved and get copied endlessly by other projects.

What about Flux? They don't say it was used for training, it wasn't, there are bugs with it that break cudagraphs or similar that aren't that impactful. On the other hand, it uses CLIP reference, and CLIP reference is buggy, so this is buggy...

liuliu 9 hours ago [-]
Congrats on finding a bug!

However, the keyword here is training / inference divergence. Unfortunately, nobody is going to spend multi-million to retrain a model, so our reimplementation needs to be bug-to-bug correct to use the trained weights properly. That's why the reference implementations are essential because it is from the original model trainers so you have the best "bet" on matching the training code properly.

To give you some concrete example of bugs we needs to maintain:

1. In SDXL, they use OpenClipG for text encoding, but wrongfully uses 0 as padding tokens (corresponding to symbol "!") whereas even for OpenClipG its own training, the endoftext token was used as padding token. However, if you switching SDXL to use endoftext token as padding token, due to training / inference divergence, you get subpar generated images.

2. In FLUX, we mainly use T5 as text encoder. However, T5 usually used as encoder with mask to exactly the same input length, to avoid extended impact of padding tokens. In FLUX, we don't apply mask for T5 text encoding, hence intuitively causing padding token to take more effect than it should. Again, "fixing" this bug without retraining you will get subpar generated images.

There are many examples like this, some are easier to fix some are not (HiDream uses a different ODE solver that is different than what we usually do for rectified flow, hence you need to negate its prediction to be compatible with existing samplers, but this is "easier to fix").

TL;DR: Yes, there are bugs in software, but we better to maintain bug-to-bug compatibility than trying to "fix" it, hence highlight the importance of a "done" reference implementation, rather than a usual "active" implementations in software industry otherwise.

(I maintain the most complete reimplementation of SoTA media generation models in Swift: https://github.com/drawthingsai/draw-things-community/tree/m.... So I tend to think that I know one or two about "reimplementation from scratch".)

doctorpangloss 7 hours ago [-]
I think if you read the issue carefully you would understand that the CLIP implementation in transformers and as published by OpenAI is wrong and does not match their trained model code; and that doing the fix I suggest, empirically for me and in theory, improves results.
42lux 14 hours ago [-]
You can disable clip l on flux without a loss in quality. You are also making an elephant out of a fly. CLIP is used everywhere.
doctorpangloss 9 hours ago [-]
Consider another interpretation: CLIP L in Flux can be disabled without a loss in quality because the way it is used is buggy!
electroglyph 11 hours ago [-]
It shouldn't take a lot of effort to fix a tokenizer...
doctorpangloss 7 hours ago [-]
People are a little too blinded by the insight porn of matching buggy behavior to just read and comprehend the issue. They can’t engage with the simpler and more pornographic insight porn that the reference implementations are buggy and do not match the trained artifacts.
reedlaw 17 hours ago [-]
I'm not sure what this means. If it means the Stable Diffusion 3.5 model, why is it fetching that here: https://github.com/yousef-rafat/miniDiffusion/blob/main/enco...

The training dataset is very small, only including fashion-related pictures: https://github.com/yousef-rafat/miniDiffusion/tree/main/data...

yousef_g 17 hours ago [-]
The dataset is for trying out fine-tuning of the diffusion model. It's a reimplementation of SD3 by writing the code from scratch again, but the weights are taken from HuggingFace due to hardware constraints on my part.
jatins 5 hours ago [-]
> It's a reimplementation of SD3 by writing the code from scratch again, but the weights are taken from HuggingFace due to hardware constraints on my part.

Could you clarify what you mean by this part -- if the weights are taken from HF then what's the implementation for?

MoonGhost 2 hours ago [-]
My guess the weights from HF are used as initial state for the model because full training is too expensive. Then small dataset is used to train in further for short time. Which is fine tuning. Together it shows that model is 1) compatible 2) trainable. In theory it can be trained from scratch on big dataset. I didn't look in the code yet so the questions are: 1) can it be trained in parallel? 2) resources required for training?

Anyway, I may try to train it on limited specialized dataset...

montebicyclelo 30 minutes ago [-]
> if the weights are taken from HF then what's the implementation for

The weights are essentially a bunch of floating point numbers, (grouped into tensors). The code says what operations to do with the weights. E.g. say you load matrix W from the weights, you could do `y = W @ x`, or `y = W.T @ x`, or `y = W @ W @ x` etc.

reedlaw 17 hours ago [-]
So this implements SD3 inference and fine-tuning?
Dwedit 11 hours ago [-]
Does using pure PyTorch improve performance on non-NVIDIA cards in any way? Or is PyTorch so highly optimized for CUDA that no other GPU vendors have a chance?
jwitthuhn 10 hours ago [-]
Pytorch also runs great on apple silicon, though it is hard to directly compare because Apple's high end GPUs can't compute anywhere near as much as nvidia's high end stuff.

e: I'll also add that pytorch does still have one oddity on apple silicon which is that it considers each tensor to be 'owned' by a particular device, either a cpu or gpu. Macs have unified memory but pytorch will still do a full copy when you 'move' data between the cpu and gpu because it just wasn't built for unified memory.

brcmthrowaway 10 hours ago [-]
Does pytorch work on AS out of the box? Or do you need some apple specific package
thom 7 hours ago [-]
`uv pip install torch` just works, set your default device to `mps:0`, enjoy the RAM. Depends what you're doing though - some stuff isn't implemented, so if you're trying to fit a Gamma/Beta/Student-T distribution you're out of luck.
chickenzzzzu 10 hours ago [-]
It is possible to run ML workloads on for example AMD devices via Vulkan. With newer extensions like cooperative matrix, and maybe also in the future some scheduling magic exposed by the driver through a new extension, the remaining single digit percent gap CUDA has will evaporate.
3abiton 10 hours ago [-]
It seems to be the case, although pytorch rocm is coming around slowly. Very slowly, if you get it working that is.
VeejayRampay 10 hours ago [-]
I believe pytorch works nicely with rocm, but I don't know if it's nicely to the point where it's "on par"
albert_e 18 hours ago [-]
Sounds like a great resources for learners.

Just wondering aloud --

Is there a tutorial/explainer by any chance that a beginner could use to follow along and learn how this is done.

an0malous 17 hours ago [-]
fast.ai has a course on building Stable Diffusion: https://course.fast.ai/Lessons/part2.html
BinaryMachine 15 hours ago [-]
Great resource Jeremy Howard is awesome, I have been waiting to take this course and follow along because anything older than a year in Deep Learning is already outdated. I hope they release a new version.
whiplash451 12 hours ago [-]
I don’t think this is true. The fast.ai class covers a lot of fundamentals that are still valid and useful today.
godelski 9 hours ago [-]

       self.q = nn.Linear(embed_size, embed_size, bias = False)
       self.k = nn.Linear(embed_size, embed_size, bias = False)
       self.v = nn.Linear(embed_size, embed_size, bias = False)
Try

       self.qkv = nn.Linear(embed_size, 3*embed_size, bias = False)

    def forward(...):
       ...
       qkv = self.qkv(x)
jszymborski 9 hours ago [-]
This adds connections between the parameters of q, k, and v whereas the original doesn't, unless my very tired brain is missing something.
smus 7 hours ago [-]
Nope, they all depend on x and the same is true in this scenario
godelski 4 hours ago [-]
It is actually really common practice. It is a single linear layer so there's no connection intranodes. The reason to do this is because it is a bit less computationally intensive.

tldr: linear layers have an associative property

NoelJacob 16 hours ago [-]
So, that's Stable Diffusion without license constraints, is it?
Sharlin 15 hours ago [-]
No, the inference/training algorithms, being math, are not copyrightable. OP just wrote another implementation. What's copyrighted are the models, which OP did not train from scratch (having neither the training material nor the compute to do that).
Zambyte 11 hours ago [-]
> What's copyrighted are the models

Has this actually been tested yet? Or are we still at the stage of AI companies trying to pretend this into reality?

bravesoul2 8 hours ago [-]
If the models are copyright protected then presumably they obeyed license on the upstream dependencies they included (i.e. the training data).
bredren 3 hours ago [-]
Is upstream dependency licensure necessary to establish copyright? For example, I Need a Haircut was still a unique work regardless of the rights to sample Alone Again.
dheera 10 hours ago [-]
I mean, if you take a match to a blank CD-ROM, or shoot neutrinos at a USB drive, there is a very small chance that you get the SD weights stored on them
Zambyte 5 hours ago [-]
You can say that about literally any digital information. This isn't really interesting in the context of the copyright status of AI models.
bravesoul2 8 hours ago [-]
Oh :( wasn't what I thought it would be. Wondered why it wasn't more blown up on HN!
echelon 14 hours ago [-]
We should be specific when we say "models".

The code outlining the network vs. the resultant weights. (Also vs. any training, inference, fine tuning, misc support code, etc.)

The theoretical diagram of how the code networks and modules are connected is math. But an implementation of that in code is copyrightable.

Afaik, the weights are still a grey area. Whereas code is code and is copyrightable.

Weights are not produced by humans. They are the result of an automated process and are not afforded copyright protection. But this hasn't been tested in court.

If OpenAI GPT 4o weights leak, I think the whole world could use it for free. You'd just have to write the code to run them yourself.

bravesoul2 8 hours ago [-]
I use model architecture for the code/math and weights for the weights to avoid confusion!

Then there are hyperparameters which are also needed to be known to use the weights with the model architecture.

MoonGhost 2 hours ago [-]
> I use model architecture for the code/math

Code is copyrightable and math is not. What about 'architecture'?

vrighter 12 hours ago [-]
which means he is still in full violation of their license
ineedasername 11 hours ago [-]
When I think of SD 3.5 (or any version) I think of the portion that results from training, i.e., the weights. The code seems less important? I mean as far as output quality is concerned, or performance. But I'm honestly not sure, and not trying to judge these efforts on that basis.
refulgentis 14 hours ago [-]
I'm embarrassed to ask: can someone elaborate on, say, what we have now that we didn't have before the repo existed?

I have studiously avoided making models, though I've been adjacent to their output for years now... I think the root of my confusion is I kinda assumed there was already PyTorch based scripts for inference / training. (I assumed _at least_ inference scripts were released with models, and kinda figured fine-tuning / training ones were too)

So then I'm not sure if I'm just looking at a clean room / dirty room rewrite of those. Or maybe everyone is using "PyTorch" but it's usually calling into CUDA/C/some proprietary thingy that is much harder to grok than a pure PyTorch impl?

Anyways, these arent great guesses, so I'll stop myself here. :)

rockemsockem 10 hours ago [-]
I believe this is the main piece

> with minimal dependencies

I haven't tried running SD 3.5 specifically, but it's built on hugging face libraries which I personally always find to be a mess of dependencies that make it really hard to setup without the exact configuration the original developers used (which is often not provided in enough detail to actually work). This makes it pretty hard to run certain models especially if it's a few months/years after the original release.

For example this appears to be the requirements for the stability AI reference implementation for SD3.5 and there are no versions specified and it includes "transformers" which is just an enormous library.

https://github.com/Stability-AI/sd3.5/blob/main/requirements...

refulgentis 10 hours ago [-]
Ah, tyvm, that maps well onto my knowledge set, I have a ONNX inference wrapper written in Dart. However, I have never been able to leverage transformers.js ONNX demo code, i.e. have a reference to port to Dart.

IIRC it is written in an abstraction layer that supports a transformers-like API surface. This also makes it opaque to figure out what you're actually passing to the model, adding a Python dep mess on top of that...woo boy.

_tqr3 12 hours ago [-]
Stability AI, creators of Stable Diffusion models release their products under own Stability AI Community License which is not "free" like MIT license. You are not allowed to modify the weights in certain ways.

This package is basically running the model (inference) and maybe fine tuning it using existing AI weights. A great way to learn but still could run into same licensing issue.

refulgentis 12 hours ago [-]
You can't finetune SD 3.5!?

I thought the community license stuff was about keeping people from using it in prod and charging for it without Stability getting at least a small taste.

This sucks.

I haven't been keeping up with gooner squad on Civit, but I did have some understanding SD was less popular, but I thought it was just because 3.5 came far too long after Flux with too little, if any, quality increase to be worth building new scaffolding for.

fc417fc802 5 hours ago [-]
> You can't finetune SD 3.5!?

They don't want you finetuning it in specific ways that might make them look bad by association.

caycep 16 hours ago [-]
How usable is the original academic source available from Ludwig Maximilian University CompViz group?
b0a04gl 18 hours ago [-]
does the DiT here actually capture cross-token attention the same way as full SD 3.5 or is it simplified for clarity?
vergessenmir 16 hours ago [-]
Is there any notable properties of this implementation, are some parts slower, faster etc
18 hours ago [-]
eapriv 16 hours ago [-]
I find it hilarious that “from scratch” now somehow means “in PyTorch”.
monsieurbanana 15 hours ago [-]
If any "from scratch" post doesn't start with linking to a Primitive Technology video, I'm closing the tab
mkoubaa 13 hours ago [-]
Unless the author was raised by chimps I'm out
0cf8612b2e1e 12 hours ago [-]
Not fusing heavier elements from hydrogen? I’m out.
chairmansteve 13 hours ago [-]
Yeah. Should have done it in assembly.
mardifoufs 11 hours ago [-]
Pytorch is a pretty basic building block when you get to some degree of model complexity. It wouldn't really be interesting to implement autograd or some other things pytorch provides imo when the goal is to show a reimplantation of something as "higher" level as SD. It's similar to how I don't mind it when someone doesn't reimplement an OS, or a JavaScript engine when writing a web app from scratch.

And there's been a recent surge in abstractions over pytorch, and even standalone packages for models that you are just expected to import and use as an API (which are very useful, don't get me wrong!). So it's nice to see an implementation that doesn't have 10 different dependencies that each abstract over something pytorch does.

eapriv 2 hours ago [-]
> It wouldn't really be interesting

Andrej Karpathy did exactly that, and I think it’s quite interesting.

nothrowaways 7 hours ago [-]
Pure pytorch?
squircle 18 hours ago [-]
Although I'm leaning heavily away from being passionate about software development, this is a cool project, and its freaken awesome how anyone can now reinvent the wheel from first principles.
CamperBob2 17 hours ago [-]
Add a Hugging Face Token in get_checkpoints.py before running the script.

Can you be a bit more specific here? It's not clear what such a token is, what it takes to get one, or where it would be placed in get_checkpoints.py.

einsteinx2 16 hours ago [-]
> what such a token is

An API token from Hugging Face

> what it takes to get one

You generate them in your Hugging Face account

> where it would be placed in get_checkpoints.py.

Line 59 in the empty quotes where it says token = “”

Dwedit 11 hours ago [-]
Leaving off the "API" part from "API Token" causes confusion, since AI models tokenize all text into "tokens" before running the model. It's using the same word to describe two very different things.
einsteinx2 11 hours ago [-]
Yep totally. Fwiw I’m not part of the project I just saw the comment and figured I’d try and help.
CamperBob2 15 hours ago [-]
Ah, I see it now, thanks.

That's the kind of thing that, stylistically speaking, it's good to define at the very top of the module.

einsteinx2 11 hours ago [-]
Agreed. I’m not part of the project I just saw your comment and figured I’d try and help.
hkon 14 hours ago [-]
now do it in minecraft
SV_BubbleTime 9 hours ago [-]
[flagged]
theturtle 17 hours ago [-]
Cool. Can it still make images of Anne Hathaway leading a herd of blue giraffes on the Moon?
IncreasePosts 14 hours ago [-]
Seems difficult, as there are no known portraits of Anne Hathaway