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

question about the basic_rnn_seq2seq (the untied architecture) #61

Open
Yutasq opened this issue Sep 7, 2019 · 0 comments
Open

question about the basic_rnn_seq2seq (the untied architecture) #61

Yutasq opened this issue Sep 7, 2019 · 0 comments

Comments

@Yutasq
Copy link

Yutasq commented Sep 7, 2019

Hi una-dinosauria,

Thank you for your code. I have a little question about your code (line 135-136 in seq2seq_model.py), based on my understanding, it seems that the same cell parameters are used for the encoder and decoder in the untied mode? Should we copy the cell e.g. copy.deepcopy(cell) as written in the basic_rnn_seq2seq that posted by tensorflow, and employ it in the rnn_decoder?

Thanks.

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

1 participant