comfy_mtb icon indicating copy to clipboard operation
comfy_mtb copied to clipboard

[feat] remove nodes

Open melMass opened this issue 1 year ago • 2 comments

Describe the problem

  • [ ] Remove I/O nodes used in VHS, initially forked from mtb it does much more now. (this will cut out A LOT of js code)
  • [ ] Image Resizing nodes / Number nodes etc. Most people use essentials anyway
  • [ ] FaceSwap / FaceRestore / FILM

Describe the solution you'd like

If not too complex, add a handler script to convert flows using these deleted nodes automatically

Alternatives considered

Deprecate MTB completely

Additional context

No response

melMass avatar Apr 10 '24 12:04 melMass

  • [x] remove the multi constant attempt, PR on comfy are hard to get merged so I won't have features relying on that from now on. These nodes won't work without the PR and it's currently confusing

https://github.com/comfyanonymous/ComfyUI/pull/3329

melMass avatar Jun 09 '24 07:06 melMass

  • [ ] Optimize batch from history: I still did not find a better approach to this and I use it all the time but it's clunky and could be optimized:

    • As the history grow the execution slowdown significantly, a solution is to clear the history but that's not ideal.
    • The host IP resolver doesn't always work from remotes.
    • It only support a single OUTPUT_NODE that's probably its biggest limitation currently

    Solutions/Ideas:

    • As for the WIP playlist nodes, it should maybe rely on disk and manage its history, that would allow greater control

melMass avatar Jul 02 '24 11:07 melMass