onediff icon indicating copy to clipboard operation
onediff copied to clipboard

Once used, Model Speedup cannot be deactivated

Open thefoxfarmer opened this issue 1 year ago • 2 comments

Describe the bug

Once a sampling workflow has used the Model Speedup node once, removing the node from the workflow does not have the effect of removing the change to the sampling process.

Your environment

macOS 14.5

OneDiff git commit id

4b35dc2a8d53c3122862b51cc69c66f59738b407

OneFlow version info if you have installed oneflow

(venv) (floppy onediff/ 0 5177)$ python -m oneflow --doctor
libibverbs not available, ibv_fork_init skipped
path: ['/home/ff/ComfyUI/venv/lib/python3.10/site-packages/oneflow']
version: 0.9.1.dev20240720+cu118
git_commit: dcaba7d
cmake_build_type: Release
rdma: True
mlir: True
enterprise: False

How To Reproduce

  1. Insert the Model Speedup node into a working workflow.
  2. Execute the workflow.
  3. Remove the Model Speedup node from the workflow.
  4. Execute the workflow again.

The complete error message

There is no error message. The effect of the Model Speedup node continues, though the node is gone.

Additional context

This would seem like something that one couldn't complain about if this were perfect. (Why remove it?) I don't know how close to perfect it is, but I know it is not perfect, so I need to be able to turn it off and I would prefer not to have to restart ComfyUI to do that. (Even if it did seem to be perfect, I'd want to turn it off to A/B test it now and then.)

thefoxfarmer avatar Jul 21 '24 04:07 thefoxfarmer

@ccssu Maybe know about this

strint avatar Jul 22 '24 08:07 strint

Please set the inplace parameter to False ,then remove this node from the workflow, and its impact on the sampling process will be removed. @thefoxfarmer

image

ccssu avatar Jul 23 '24 02:07 ccssu