Fix bug with scheduler stepping twice instead of once

fastai / diffusion-nbs

Getting started with diffusion

Apache License 2.0

628 stars 281 forks source link

Fix bug with scheduler stepping twice instead of once #49

Closed kevinji closed 7 months ago

kevinji commented 7 months ago

Fixes #42.

review-notebook-app[bot] commented 7 months ago

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

kevinji commented 7 months ago

@pcuenca are you able to review this?

johnowhitaker commented 7 months ago

@kevinji I think you're right - I should do some archaeology to see what the diffusers code used to do, but this seems like an improvement to the original code anyway. Thanks :)

pcuenca commented 7 months ago

Thanks @kevinji!