-
Notifications
You must be signed in to change notification settings - Fork 152
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[ISSUE] Batching Video and Frames with 'No Face' cause unrecoverable error. #153
Comments
There's a PR which may fix the issue, I just honestly haven't had the time (or interest) for this project at all, would help us all if you want to test it: #115 |
Happy to test it, is it ready to go and do you need to resolve your conflicts?
BR.
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Jukka Seppänen ***@***.***>
Sent: Sunday, November 3, 2024 11:06:09 AM
To: kijai/ComfyUI-LivePortraitKJ ***@***.***>
Cc: William Russell ***@***.***>; Author ***@***.***>
Subject: Re: [kijai/ComfyUI-LivePortraitKJ] [ISSUE] Batching Video and Frames with 'No Face' cause unrecoverable error. (Issue #153)
There's a PR which may fix the issue, I just honestly haven't had the time (or interest) for this project at all, would help us all if you want to test it: #115<#115>
—
Reply to this email directly, view it on GitHub<#153 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BI22TKHAUXYAVWAV2ZK7LOTZ6ZX2DAVCNFSM6AAAAABRC7B7GOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGU2DGMZSGM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Just following up, it works perfect, no issues that I can tell that didn't exist before, face outside of frame, morphing, etc. but regardless, does not throw errors and breezes through.
Thanks a bunch, I will update git as well with a comment.
Cheers Friend
Cruel_world
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Jukka Seppänen ***@***.***>
Sent: Sunday, November 3, 2024 11:06:09 AM
To: kijai/ComfyUI-LivePortraitKJ ***@***.***>
Cc: William Russell ***@***.***>; Author ***@***.***>
Subject: Re: [kijai/ComfyUI-LivePortraitKJ] [ISSUE] Batching Video and Frames with 'No Face' cause unrecoverable error. (Issue #153)
There's a PR which may fix the issue, I just honestly haven't had the time (or interest) for this project at all, would help us all if you want to test it: #115<#115>
—
Reply to this email directly, view it on GitHub<#153 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BI22TKHAUXYAVWAV2ZK7LOTZ6ZX2DAVCNFSM6AAAAABRC7B7GOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGU2DGMZSGM>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
I'm sure you are aware, just starting a thread for this specific issue. "Live Portrait Retargeting" as well as just using "Live Portrait Process" by itself, is a bit of a kluge when it comes to batching video.
"Live Portrait Retargeting" just fails if there is a missing Face in frame passed from the Cropper.
"Live Portrait Process" (Without Retargeting so it will work) will "Smooth" the frames so if there are a few missing, you get a fuzballblur. (Its an industry term) For the whole batch.. not just the missing frame..
As I said, Im sure you are aware - but if there was anything you would do about it I would be grateful.
Thanks Bunches..
The text was updated successfully, but these errors were encountered: