Skip to content

Conversation

chaObserv
Copy link
Contributor

@chaObserv chaObserv commented Jun 30, 2025

This will change the results in RF.

Unlike other SDE samplers, the current ER-SDE appears compatible with RF because it does not introduce as much stochasticity. However, this still causes the results to be slightly over-bright. Moreover, it does not properly support reverse-time SDEs under RF. Since ER-SDE unifies reverse-time SDEs and ODEs, using its VP algorithm to apply alpha scaling may be more appropriate.

To provide access to reverse-time SDE and ODE solver types, this PR also introduces a custom sampler node.

Test with SD3.5 medium, CFG 4.0, and sgm_uniform 20 steps.
All images in the Euler column are the same and serve as the baseline reference.
er_sde_update_grid_resized

With the current ER-SDE, the cloud loses detail and appears as plain white light in this case.

Apply alpha scaling in the algorithm for reverse-time SDE and add custom ER-SDE sampler node for other solver types (SDE, ODE).
@comfyanonymous comfyanonymous merged commit b22e97d into comfyanonymous:master Jul 1, 2025
5 checks passed
@chaObserv chaObserv deleted the er_sde_update branch July 3, 2025 07:47
rakki194 pushed a commit to rakki194/ComfyUI that referenced this pull request Jul 21, 2025
…yanonymous#8744)

Apply alpha scaling in the algorithm for reverse-time SDE and add custom ER-SDE sampler node for other solver types (SDE, ODE).
rakki194 pushed a commit to rakki194/ComfyUI that referenced this pull request Jul 29, 2025
…yanonymous#8744)

Apply alpha scaling in the algorithm for reverse-time SDE and add custom ER-SDE sampler node for other solver types (SDE, ODE).
rakki194 pushed a commit to rakki194/ComfyUI that referenced this pull request Jul 29, 2025
…yanonymous#8744)

Apply alpha scaling in the algorithm for reverse-time SDE and add custom ER-SDE sampler node for other solver types (SDE, ODE).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants