-
Notifications
You must be signed in to change notification settings - Fork 5
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
Reproduce results on SST-2 / SST-5 #2
Comments
did you solve this problem? this codebase is too big for me to read. |
Hi, very sorry we did not have time to clean up the codes. As in shown in the instruction, please follow the configuration |
Hi Nguyen,
Thank you for taking the time to reply to my email, I have two more questions:
1. There are some letters representing dimensions in your code, such as tq, nq, etc., whether tq means t*q? Or there is a document explaining these parameters uniformly?
2. What's the difference between 'nstack2seq' and 'nstack_merge2seq'? I see that the encoders and decoders of the two are different, but more because the code base is too large, I really can’t figure it out.
…------------------ 原始邮件 ------------------
发件人: "nxphi47/tree_transformer" ***@***.***>;
发送时间: 2021年3月26日(星期五) 下午5:34
***@***.***>;
***@***.******@***.***>;
主题: Re: [nxphi47/tree_transformer] Reproduce results on SST-2 / SST-5 (#2)
Hi, very sorry we did not have time to clean up the codes. As in shown in the instruction, please follow the configuration dwnstack_merge2seq_node_iwslt_onvalue_base_upmean_mean_mlesubenc_allcross_hier to find its implementation in the files nstack_archs.py and nstack_transformer.py
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
could you please provide an instruction on how to reproduce your results on SST.
It's impossible for me to navigate in your impressive codebase (its huge). E.g. 8 different attention implementations:
2. dptree_individual_multihead_attention.py
3. dptree_multihead_attention.py
4. dptree_onseq_multihead_attention.py
5. dptree_sep_multihead_attention.py
6. nstack_merge_tree_attention.py
7. nstack_tree_attention.py
8. nstack_tree_attention_eff.py
Thanks!
The text was updated successfully, but these errors were encountered: