A node that is loading weights looks identical to one that is computing:
progress_state carries only state/value/max, and load_models_gpu runs inside
the sampler, so the UI shows the sampler sitting at 0% while weights move.
Adds an optional activity field to the node progress state, reported through a
hook like the existing progress bar hook, and sets it to "loading" around the
VRAM load. The field is absent unless something sets it, so existing clients
are unaffected.
* model_prefetch: add friendly API to pause comfy compiler
* sparse_attention: Pause comfy compiler for un-freed pieces
The VSA plan and pool stats do not free within the transformer causing
a comfy-compiler rogue allocation and graph break.
Ideally these should be freed in the lifetime of the transformer run
(and comfy-compiler scope) but there isnt a convenient place for the
particular lifecylce intended by these variables. So go with the pause()
approach.
This is the result of an experiment where I asked a LLM to create a better
sampler for the base Anima model and gave it a lot of different things to
try.
I wouldn't call this revolutionary but from my basic testing it seems to be
a bit better than the other samplers at lower steps so I have decided to
include it in ComfyUI.
For best result use it at CFG 2.0 on base Anima. It should also work on
other models but I'm not sure about the optimal parameters.
* fix: respect user directory for default database
* refactor: use None default for --database-url instead of argv scan
Per review: default --database-url to None and treat a non-None value
as explicit at resolution time. Removes the sys.argv scan and the
database_url_explicit attribute. database_default_path now serves
directly as the legacy copy source. Adds regression tests for the
unchanged no-flag default path and explicit URLs at the legacy
location.
* refactor: rename legacy database to .bak after copy
Per review: after copying the legacy install-dir database to the
effective user directory, rename the original to comfyui.db.bak so a
later launch without --user-directory cannot silently fall back to a
diverged copy, while keeping the file around for recovery. Also hoist
the database_default_path import to module scope.
* fix: guard legacy migration on existing .bak and rename before copy
Per review: bail out of the legacy migration when comfyui.db.bak
already exists, so only the first run migrates and later launches with
a fresh --user-directory cannot grab a database another instance is
using. Rename before copy so os.replace fails fast if the legacy DB is
held open by a running instance.
---------
Co-authored-by: guill <jacob.e.segal@gmail.com>
Vinh Trinh