-
Notifications
You must be signed in to change notification settings - Fork 40
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
Why don't use CLIP text encoder as the text encoder for text-motion evaluator? #43
Comments
Hi, CLIP text encoder could be a better choice now. However, it was not
that common to use CLIP at that time.
…On Thu, 30 May 2024 at 23:01, Mingshuang Luo ***@***.***> wrote:
Hi, @EricGuo5513 <https://github.com/EricGuo5513> , about the text-motion
evaluator, I want to know why you don't use the CLIP text encoder as the
text encoder for text-motion evaluator. I think it is much easier if using
the CLIP text encoder directly.
—
Reply to this email directly, view it on GitHub
<#43>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKRYNB357IEZHIX7UV3HNTLZE7R2BAVCNFSM6AAAAABISACADWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGMZDMOBXGEZDCMQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Oh, thanks for your reply. I think if we change to use CLIP text encoder, maybe we will have a new comparison between different methods. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, @EricGuo5513 , about the text-motion evaluator, I want to know why you don't use the CLIP text encoder as the text encoder for text-motion evaluator. I think it is much easier if using the CLIP text encoder directly.
The text was updated successfully, but these errors were encountered: