Skip to content

Conversation

@chaObserv
Copy link
Contributor

Noise decomposition

The current noise decomposition seems right for the default r, but not for other cases. For instance, setting r=1 in SEEDS-2 will fully use a new noise, instead of depending on the noise from the first stage that already covers the current interval [s, t]. This seems not to match the claim that "overlapping time intervals need to be dependent."

Therefore, I tried to follow the existed derivation to obtain another result.
圖片
In this decomposition, when r=1, z1 dominates the current interval. In SEEDS-2, this appears to produce relatively sharper results at low CFG and in the low NFE regime when r > 0.5.

Test with V-prediction ztsnr model and simple scheduler.

  • SEEDS-2 CFG 1
seeds_2_cfg1_grid_00001_resized
  • SEEDS-2 CFG 3
seeds_2_cfg3_grid_00001_resized
  • SEEDS-3 CFG 1
seeds_3_cfg1_grid_00001_resized

Other changes

  • Extract phi functions since they are commonly used in exponential-based samplers (DPM-solver, RES). The function names follow UniPC's h_phi_k variable.
  • Cleanup and rearrangements.

- Update the decomposition to reflect interval dependency
- Extract phi computations into functions
- Use torch.lerp for interpolation
@comfyanonymous comfyanonymous merged commit 32a627b into comfyanonymous:master Aug 31, 2025
6 checks passed
@chaObserv chaObserv deleted the seeds_update branch September 1, 2025 06:51
Thor-ATX pushed a commit to asteriafilmco/ComfyUI that referenced this pull request Sep 3, 2025
…e-update

* commit '4f5812b93712e0f52ae8fe80a89e8b5e7d0fa309': (77 commits)
  Update template to 0.1.73 (comfyanonymous#9686)
  ImageScaleToMaxDimension node. (comfyanonymous#9689)
  Accept prompt_id in interrupt handler (comfyanonymous#9607)
  uso -> uxo/uno as requested.  (comfyanonymous#9688)
  USO style reference. (comfyanonymous#9677)
  Enable Convolution AutoTuning (comfyanonymous#9301)
  Implement the USO subject identity lora. (comfyanonymous#9674)
  Probably not necessary anymore. (comfyanonymous#9646)
  SEEDS: update noise decomposition and refactor (comfyanonymous#9633)
  convert Primitive nodes to V3 schema (comfyanonymous#9372)
  convert nodes_stability.py to V3 schema (comfyanonymous#9497)
  convert Video nodes to V3 schema (comfyanonymous#9489)
  convert Stable Cascade nodes to V3 schema (comfyanonymous#9373)
  ComfyUI version 0.3.56
  Lower ram usage on windows. (comfyanonymous#9628)
  ComfyUI v0.3.55
  Update template to 0.1.70 (comfyanonymous#9620)
  Trim audio to video when saving video. (comfyanonymous#9617)
  Support the 5B fun inpaint model. (comfyanonymous#9614)
  Support wan2.2 5B fun control model. (comfyanonymous#9611)
  ...
toxicwind pushed a commit to toxicwind/ComfyUI that referenced this pull request Oct 12, 2025
- Update the decomposition to reflect interval dependency
- Extract phi computations into functions
- Use torch.lerp for interpolation
adlerfaulkner pushed a commit to LucaLabsInc/ComfyUI that referenced this pull request Oct 16, 2025
- Update the decomposition to reflect interval dependency
- Extract phi computations into functions
- Use torch.lerp for interpolation
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