Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to transfer JSON between ComfyBox & ComfyUI #129

Open
WinstonWoof opened this issue Jul 24, 2023 · 7 comments
Open

Unable to transfer JSON between ComfyBox & ComfyUI #129

WinstonWoof opened this issue Jul 24, 2023 · 7 comments

Comments

@WinstonWoof
Copy link

image

It loads & runs its own default plus https://civitai.com/models/111463?modelVersionId=124584 ) but it doesnt load up any existing JSON or PNG files from my local PC either via the Load Button or by dragndrop

Conversely I cant load any ComfyBox Json (or generated images) in ComfyUI (### ComfyUI Revision: 1197 [67be7eb8])

@Katmoget
Copy link

similar issue, I am unable to load any ComfyBox files in ComfyUI

@DarkAlchy
Copy link

I am beginning to think this is a dead project and really only made for Linux to begin with.

@Katmoget
Copy link

Katmoget commented Aug 3, 2023

yes, it certainly looks like it at the moment, it's a shame it's pretty good otherwise.

@hobojoker
Copy link

hobojoker commented Aug 7, 2023

Conversely I cant load any ComfyBox Json (or generated images) in ComfyUI (### ComfyUI Revision: 1197 [67be7eb8])

Was this ever intended to be a feature? ComfyUI .json will transfer and be used in ComfyBox because comfybox uses ComfyUI as a backend, but Comfybox dramatically changes the .json so I wouldn't expect ComfyBox .json to work in ComfyUi. It's only ever indicated as being one-way.

@Katmoget
Copy link

I guess you're right, working as designed. it would be handy as a feature though.

@Lalimec
Copy link

Lalimec commented Aug 23, 2023

did you guys tried comy paste, the issue i have is all the input fields in the pasted nodes being turned into a ui element. i want some values to be predetermined but wasnt able to do it automatically.

@space-nuko
Copy link
Owner

I am beginning to think this is a dead project and really only made for Linux to begin with.

Sorry about the lack of communication, for reasons outside of my control I've recently become absurdly busy with real life and haven't had much time to work on personal projects as much. But if I get some time in the future I intend to fix some of the open issues

The workflow importing feature is kind of WIP because ComfyBox has a different way of saving node data, I did a bit of work to try to improve the feature parity between ComfyBox and ComfyUI recently: comfyanonymous/ComfyUI#1310

Also there isn't yet a way to convert a ComfyBox workflow back into ComfyUI since that would be more involved (how to deal with nodes in ComfyBox but missing in ComfyUI for example), I don't think such a feature would be impossible but there would have to be some compromises made, like you'd have to rebuild the missing parts of the workflow yourself in a way that makes better sense for ComfyUI. And unless ComfyUI supports node events the frontend logic feature of ComfyBox cannot be ported yet

I actually develop and run this project on Windows with pnpm, I'll have to clarify the installation instructions since a bunch of people seem confused on how to install. The part that requires the command line isn't actually necessary if all you want to do is run the program since there is a precompiled version in the README (just unzip and run)

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

No branches or pull requests

6 participants