-
Notifications
You must be signed in to change notification settings - Fork 25
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
Unexpectedly high (over 10%) search performance #18
Comments
I know #4.
But I already use v1.1, so it may have more 5% gain. |
@nobuyuki-nc Hi, thanks for sharing your experiment.
|
Thank you for your reply. I agree with your comment. Thanks |
@nobuyuki-nc Here is my result of
|
Thank you, it's an interesting result! |
How would you interpret the results of this experiment? The effect of bs120 is better than that of bs640, which is very rare in comparative study.Is it because the optimizer is different? |
Because one of the improvements I made to your baseline was to make the results more similar across batches, but based on the results you posted, it's a bit confusing for me. |
Here, the main point is that the overall performance is higher than reported in the paper. On the effect of batch-size, the new results with Adam are 70.30% (1s exact, bsz=640) and 68.60% (1s exact, bsz=120). Again, bsz=640 is better than bsz=120. There's no result that contradicts the argument made in the paper. You may observe bsz=120 can perform better sometimes with different query lengths. But this can happen because our model is only trained to maximize 1s-segment-similarity, not the sequence similarity. |
I'm tring to reproduce your paper results.
In my resource, I can use only TR_BATCH_SZ=120.
So your paper said top-1 exact match rate 55.9% and near match rate 62.3% at query length 1s in Table 3.
But I get 67.95% of exact match rate and 73.0% of near match rate. (see below)
Is it an applicable result?
I use dataset-full v1.1 from ieee-dataport.org.
I change configuration TEST_DUMMY_DB=100k_full_icassp.
Then train & generate, evaluate
The text was updated successfully, but these errors were encountered: