diffusers icon indicating copy to clipboard operation
diffusers copied to clipboard

Add experimental Heun scheduler

Open pcuenca opened this issue 3 years ago • 1 comments

Without duplicating the sigmas and timesteps (reference: #1336)

âž• No duplication of sigmas and timesteps. Does not require ugly function index_for_timestep (or confusing oneliner repeated twice in the code)

âž– Requires modification of all pipelines. The pipeline loop is uglier.

All in all I don't think we gain any more clarity than in #1336, so I recommend we adopt #1336 unless it's important that the timesteps and sigmas are not duplicated.

This implementation does work. I'll push a fix for #1336 so it works too.

pcuenca avatar Nov 21 '22 15:11 pcuenca

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.