fastai/diffusion-nbs

Minor typo in Stable Diffusion Deep Dive notebook

daspartho opened this issue · 1 comments

daspartho commented

Hi, @johnowhitaker

In the positional embedding section at the line

But now instead of dealing with ~50 tokens we just need one for each position (77 total):

I think the number of tokens should be 50K instead of 50. Could you please confirm this?

Thanks

johnowhitaker commented

You're right that should be 50K - I'll change that along with one or two other tweaks soon.Thanks :)

  • 👍1