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

About mean_pooling on text sequence #94

Closed
celestialxevermore opened this issue Jan 3, 2023 · 2 comments
Closed

About mean_pooling on text sequence #94

celestialxevermore opened this issue Jan 3, 2023 · 2 comments

Comments

@celestialxevermore
Copy link

Dear author, I hope you enjoy your new year.

I have a quetion about, the reason why you do not use the function, _mean_pooling_for_similarity_sequence.

Is there any special reason for that?

I also looked out your previous model, UniVL, but I cannot find any reason about that.

I hope you reply soon.

Thx.

Sincerly,

@ArrowLuo
Copy link
Owner

ArrowLuo commented Jan 5, 2023

Hi @celestialxevermore, good question., but to be honest, I forget the last reason. (Maybe) When I tested the architecture, the results did not have a big gap, so I kept the same process as CLIP did finally. The _mean_pooling_for_similarity_sequence just filters the first `[CLS]' and finishes mean pooling on real tokens. Best~

@celestialxevermore
Copy link
Author

Thx!!

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