Skip to content

fixed arg *hidden_state*#20

Open
DansYU wants to merge 1 commit intondrplz:masterfrom
DansYU:patch-1
Open

fixed arg *hidden_state*#20
DansYU wants to merge 1 commit intondrplz:masterfrom
DansYU:patch-1

Conversation

@DansYU
Copy link
Copy Markdown
Contributor

@DansYU DansYU commented May 6, 2020

fixed: hidden_state != None;
changed: last_state_list

fixed: hidden_state != None;
changed: last_state_list
@DansYU
Copy link
Copy Markdown
Contributor Author

DansYU commented May 6, 2020

  • when used seq2seq, Decoder don't need to init hidden_state.
  • even if return_all_layers is True, last_state_list of return need to return whole hidden_state instead of the last layer state

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

Successfully merging this pull request may close these issues.

1 participant