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

[feat]: add sequence_model: self_attention #104

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

lgqfhwy
Copy link
Collaborator

@lgqfhwy lgqfhwy commented Dec 31, 2021

add sequence model : self attention.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


yancheng.lgq seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@lgqfhwy lgqfhwy added the ci_test_passed ci test passed label Dec 31, 2021
@lgqfhwy lgqfhwy changed the title [feat] add sequence_model: self_attention [feat]: add sequence_model: self_attention Dec 31, 2021
@github-actions
Copy link

CI Test Failed

@github-actions github-actions bot added ci_test_failed and removed ci_test_passed ci test passed labels Dec 31, 2021
@github-actions
Copy link

github-actions bot commented Jan 1, 2022

CI Test Failed

yancheng.lgq added 2 commits January 4, 2022 15:56
@github-actions
Copy link

github-actions bot commented Jan 4, 2022

CI Test Passed

@github-actions github-actions bot added ci_test_passed ci test passed and removed ci_test_failed labels Jan 4, 2022
@@ -119,4 +119,7 @@ message SeqAttGroupConfig {
optional bool tf_summary = 3 [default = false];
optional DNN seq_dnn = 4;
optional bool allow_key_search = 5 [default = false];
optional string seq_model = 6 [default = 'target_attention'];
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use enum instead

@agentai
Copy link

agentai commented Apr 1, 2022

求帮忙解决下冲突问题然后合个代码呗,比心

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci_test_passed ci test passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants