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

Results for English STS using BERT-mp #6

Open
zhujiatong628 opened this issue Mar 6, 2023 · 2 comments
Open

Results for English STS using BERT-mp #6

zhujiatong628 opened this issue Mar 6, 2023 · 2 comments

Comments

@zhujiatong628
Copy link

Hi, I'm reproducing your experiment recently, and I found when I evaluated BERT-mp for English STS. The results I got are much better than reported in the paper, so I also use your code provided at here to check. I found it's still better than paper shows. However, when I use your code to evaluate BERT-CLS for English STS, I can get the same results with the paper shows. So I was wondering is there something wrong with the results using BERT-mp for English STS?
results

@hardyqr
Copy link
Collaborator

hardyqr commented Mar 6, 2023

Hi, if you check out

parser.add_argument('--agg_mode', type=str, default="cls", help="{cls|mean|mean_std|...}")
, there are two options for mp: mean and mean_std where the first one considers padding tokens too. I used mean for reporting results which might have caused the discrepancy.

@zhujiatong628
Copy link
Author

Thanks for your reply. I also used mean, so I'm so confused about the difference.😫

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