ModelSurge / sd-webui-comfyui

An extension to integrate ComfyUI workflows into the Webui's pipeline
MIT License
514 stars 32 forks source link

this is can use api ? #115

Open rookiexjl opened 1 year ago

rookiexjl commented 1 year ago

comfyUI use api ?

ljleb commented 1 year ago

Please clarify your question. by any chance is your request for comfyui to make api requests to the a1111 webui to generate images?

rookiexjl commented 1 year ago

yes,can use api 

------------------ 原始邮件 ------------------ 发件人: @.>; 发送时间: 2023年8月10日(星期四) 晚上6:26 收件人: @.>; 抄送: @.>; @.>; 主题: Re: [ModelSurge/sd-webui-comfyui] this is can use api ? (Issue #115)

Please clarify your question. by any chance is your request for comfyui to make api requests to the a1111 webui to generate images? 请澄清您的问题。您是否要求Comfyui向A1111 WebUI发出API请求以生成图像?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

ljleb commented 1 year ago

Sorry, I do not understand.

Please forgive me if I am not correct, but it seems that you are not fluent in english. You can ask your request in the language of your preference and I will translate it online before replying back to you in english, if that is okay with you of course.

rookiexjl commented 1 year ago

是的,可以吗 用sd的api调用comfyui

ljleb commented 1 year ago

Okay, now I believe I understand. IIUC, you want an API route in the webui for the comfyui extension, that will enable setting the workflows in the comfyui accordion of the txt2img and img2img tabs, when making txt2img or img2img calls respectively. This is so that you can control which workflow types (preprocess, preprocess_latent, postprocess, postprocess_latent, etc.) will run when calling the API.

Is that correct?

For making requests to comfyui directly, you should already be able to do that by using the address where the comfyui server resides (by default, this is http://localhost:8189).

ljleb commented 1 year ago

Or sorry, not a dedicated API route, but rather allow to pass the workflows to the alwayson_scripts property?

rookiexjl commented 1 year ago

有个简单案例吗?可以发个过看看吗,感谢!

------------------ 原始邮件 ------------------ 发件人: @.>; 发送时间: 2023年8月11日(星期五) 上午10:18 收件人: @.>; 抄送: @.>; @.>; 主题: Re: [ModelSurge/sd-webui-comfyui] this is can use api ? (Issue #115)

Or sorry, not a dedicated API route, but rather allow to pass the workflows to the alwayson_scripts property? 或者抱歉,不是专用的 API 路由,而是允许将工作流传递给 alwayson_scripts 属性?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

ljleb commented 1 year ago

If you mean to ask whether there are any API at the moment, well there is NO API in the webui for this extension currently. We can add this feature in the future, if this is something you would want to use.

What would be a simple use case for you? I think there are many ways this extension could be used through the API. One way we could implement it in the future, could be something like this (note that this is not currently functioning code, I am just trying to define the feature more clearly):

hypothetical POST /sdapi/v1/txt2img

{
    "prompts": ["a majestic landscape"],
    ...
    "alwayson_scripts": {
        "comfyui": {
            "args": [{
                "id": "postprocess",
                "workflow": {...}
            }, {
                "id": "preprocess",
                "workflow": {...}
            }]
        }
    }
    ...
}

Again, this is not something that works currently, but it could work in the future when we implement it.

Is this what you want? Or is what you want to do not covered by this suggestion?

rookiexjl commented 1 year ago

谢谢

------------------ 原始邮件 ------------------ 发件人: @.>; 发送时间: 2023年8月11日(星期五) 上午10:29 收件人: @.>; 抄送: @.>; @.>; 主题: Re: [ModelSurge/sd-webui-comfyui] this is can use api ? (Issue #115)

If you mean to ask whether there are any API at the moment, well there is no API in the webui for this extension currently. We can add this feature in the future, if this is something you would want to use.

What would be a simple use case for you? I think there are many ways this extension could be used through the API. One way we could implement it in the future, could be something like this (note that this is not currently functioning code, I am just trying to define the feature more clearly): { "prompts": ["a majestic landscape"], ... "alwayson_scripts": { "comfyui": { "args": [{ "id": "postprocess", "workflow": {...} }, { "id": "preprocess", "workflow": {...} }] } } ... }
Again, this is not something that works currently, but it could work in the future when we implement it.

Is this what you want? Or what you want to do is not covered by this suggestion?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>