-
Notifications
You must be signed in to change notification settings - Fork 16
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
[BUG] in the example workflow #60
Comments
Huh. Well, I'll fix that at some point. |
Also, please, add "PCSplitSampling" to the example |
I've finally managed to properly connect "PCSplitSampling" but I also found an incompatibility with some "ComfyUI-Impact-Pack" nodes: |
I redid the example workflow. I removed some of the more obscure features from it but added eg. custom mask usage. |
I see "PCSplitSampling" right after "Load Checkpoint". Don't I have to place it at exactly before KSampler? |
@miasik I'm pretty sure it should work in either position since all it does is set a flag in |
Could you please review your example https://github.com/asagi4/comfyui-prompt-control/blob/master/workflows/example.json?raw=1
I see "PCApplySettings" is not connected
As I understand it has to be connected both to "ScheduleToCond" and "ScheduleToModel"
The text was updated successfully, but these errors were encountered: