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

Suggestion. Add a mask for face swap that is not the same as for the face detection. #27

Closed
zewolf5 opened this issue Feb 22, 2024 · 3 comments

Comments

@zewolf5
Copy link

zewolf5 commented Feb 22, 2024

For a better end result, using any of the ComfyUI tools to get a mask with only the face will be better than a square box for the face swap part.
So I tested this and works like charm. Just allow for a second mask into the "Faceswap generate" to be used for the actual faceswap. That face swap mask is too small to be used for face detection and almost always fail.

Or add a Dilation field to increase the area around the one mask being used.

image
image
image

@zewolf5
Copy link
Author

zewolf5 commented Feb 22, 2024

Ignore this. Padding = Dilation it seems.

So just close this.

@nosiu
Copy link
Owner

nosiu commented Feb 25, 2024

this might be a good idea for an attention mask for control net, adding the second mask

@nosiu
Copy link
Owner

nosiu commented Oct 20, 2024

no longer relevant, since the version update

@nosiu nosiu closed this as completed Oct 20, 2024
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