Skip to content

Loading preset workflow if no custom workflow was selected#30

Open
fylrid2 wants to merge 10 commits intonew-sankaku:mainfrom
fylrid2:main
Open

Loading preset workflow if no custom workflow was selected#30
fylrid2 wants to merge 10 commits intonew-sankaku:mainfrom
fylrid2:main

Conversation

@fylrid2
Copy link
Copy Markdown
Contributor

@fylrid2 fylrid2 commented Jun 22, 2024

This is just a short PR that makes sure the preset T2I comfyui workflow is loaded when generating images if no custom workflow was selected. I also added a preset I2I workflow but the support for that is not implemented yet so it hasn't been tested. Will look to improve this further but just wanted to get this out before I go to bed. Tried to also update base T2I settings when changing api mode but couldn't figure out a good way to do that without needing to open the window.

DEEPL translation

これは、カスタムワークフローが選択されていない場合、プリセットのT2I comfyuiワークフローが画像生成時に読み込まれるようにするための短いPRです。プリセットのI2Iワークフローも追加したが、まだ実装されていないためテストしていない。さらに改善する予定だが、寝る前にこのことを伝えたかっただけだ。apiモードを変更する際に、ベースとなるT2Iの設定も更新しようとしたが、ウィンドウを開かずに行う良い方法が見つからなかった。

@fylrid2
Copy link
Copy Markdown
Contributor Author

fylrid2 commented Jun 23, 2024

Added some extra QOL changes afterwards. Changed to use your toggle api checkbox. I previously missed that it was implemented and deprecated the old buttons. Made sure to only ping the chosen api model. Also made sure the api is immediately pinged when you change api mode to get instant feedback if you are connected or not. Fixed spelling mistakes in comments, console logs and function names from "comufy" to "comfy" for consistency. Lastly fixed so that the api dropdown menu doesn't close when toggling the api mode.

DEEPL translation

その後、いくつかのQOLの変更を追加。toggle apiのチェックボックスを使うように変更。以前は実装されているのを見落としていて、古いボタンを非推奨にしていた。選択したapiモデルのみにpingを送るようにした。また、apiモードを変更したときに、接続されているかどうかのフィードバックを即座に得るために、apiに即座にpingが送られるようにした。コメント、コンソールログ、関数名のスペルミスを "comufy "から "comfy "に修正。最後に、apiモードを切り替えてもapiドロップダウンメニューが閉じないように修正しました。

@new-sankaku
Copy link
Copy Markdown
Owner

Thank you!
I'm checking the code when I have time.
Please let me know if it's better after merging it.

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

Successfully merging this pull request may close these issues.

2 participants