Skip to content

fix(Moonvalley-API-Nodes): adjust "steps" parameter to not raise exception#12370

Merged
comfyanonymous merged 1 commit intomasterfrom
fix/api-nodes/moonvalley
Feb 10, 2026
Merged

fix(Moonvalley-API-Nodes): adjust "steps" parameter to not raise exception#12370
comfyanonymous merged 1 commit intomasterfrom
fix/api-nodes/moonvalley

Conversation

@bigcat88
Copy link
Contributor

@bigcat88 bigcat88 commented Feb 9, 2026

Default values that we have currently for all three nodes, always raising the exception:

Exception: Polling aborted due to error: Task failed: {"id": "prompt_c12b80fa-b18b-43c8-9bb4-51b3155f290e", "status": "error", "error": {"message": "Value error, warmup_steps (24) + cooldown_steps (36) must be less than steps (59)", "traceback": ""}}

Adjusted the minimal and default values ​​of the "steps" widget to work correctly.

API Node PR Checklist

Scope

  • Is API Node Change

Pricing & Billing

  • Need pricing update
  • No pricing update

If Need pricing update:

  • Metronome rate cards updated
  • Auto‑billing tests updated and passing

QA

  • QA done
  • QA not required

Comms

  • Informed Kosinkadink

@bigcat88 bigcat88 added the Core Core team dependency label Feb 9, 2026
@comfyanonymous comfyanonymous merged commit c1b63a7 into master Feb 10, 2026
17 checks passed
@comfyanonymous comfyanonymous deleted the fix/api-nodes/moonvalley branch February 10, 2026 02:58
luna-niemitalo pushed a commit to luna-niemitalo/ComfyUI that referenced this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Core team dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants