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

Bad matching generated person #890

Open
Padraix opened this issue Jan 24, 2025 · 5 comments
Open

Bad matching generated person #890

Padraix opened this issue Jan 24, 2025 · 5 comments

Comments

@Padraix
Copy link

Padraix commented Jan 24, 2025

Hello. I have a problem with the nodes from this pack. Namely, when I generate an image using the "person" option in the "UltralyticsDetectorProvider" node, the image is generated misaligned. In the "Preview Image" and "Preview Bridge (Image)" node, after KSampler node, character it generates good, although the image is cropped, but in these node it is generally ok, but in the "save image/preview image" node, which is set last in the order, the character comes out misaligned with the mask. This effect appears in the last node, which is after nodes such as: Image Resize, Edit SEG_ELT, Assemble (SEGS), SEGSPaste. So here I would look for an incorrect setting, but I don't know how to fix it. Any ideas?

Image

@Padraix Padraix changed the title Error with matching generated person Bad matching generated person Jan 24, 2025
@ltdrdata
Copy link
Owner

Please upload the workflow.

@Padraix
Copy link
Author

Padraix commented Jan 25, 2025

This is my workflow.

Generated person.json

@ltdrdata
Copy link
Owner

Image

I’m not sure why you used this approach, but SEGS Paste is a node designed to be used together with SEGS Detailer.
SEGS generated through SimpleDetector identifies each detected region along with Mask information.
When you directly apply an image to SEGS using Edit SEG_ELT, the Mask is only directly applied to the input image.

@Padraix
Copy link
Author

Padraix commented Jan 27, 2025

So how do i connect it? :) So i should connect the "Simple Detector (SEGS)" node directly to the "SEGSDetailer" node and that one to the "SEGSPaste" node?

@ltdrdata
Copy link
Owner

ltdrdata commented Jan 27, 2025

So how do i connect it? :) So i should connect the "Simple Detector (SEGS)" node directly to the "SEGSDetailer" node and that one to the "SEGSPaste" node?

Have you ever seen this workflow?
https://github.com/ltdrdata/ComfyUI-extension-tutorials/blob/Main/ComfyUI-Impact-Pack/workflow/segs-detailer.png
https://github.com/ltdrdata/ComfyUI-extension-tutorials/blob/Main/ComfyUI-Impact-Pack/workflow/segs-picker.png

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

2 participants