new-sankaku / stable-diffusion-webui-simple-manga-maker

It is an Extension feature used in the WebUI for Stable Diffusion. You can create simple comics with it.
https://new-sankaku.github.io/SP-MangaEditer/
GNU General Public License v3.0
154 stars 18 forks source link

Loading preset workflow if no custom workflow was selected #30

Open fylrid2 opened 5 months ago

fylrid2 commented 5 months ago

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 commented 5 months ago

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 commented 5 months ago

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