Skip to content
This repository was archived by the owner on Nov 22, 2022. It is now read-only.

add Boundary Pooling in BiLSTMDocAttention #256

Closed

Conversation

gardenia22
Copy link
Contributor

Summary:
Self attention Pooling introduce extra operator like ATEN while trying to export on-device model to C2.
add Boundary Pooling in BiLSTMDocAttention to use only the last hidden state as output

Differential Revision: D13789842

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jan 30, 2019
gardenia22 added a commit to gardenia22/pytext-1 that referenced this pull request Jan 30, 2019
Summary:
Pull Request resolved: facebookresearch#256

Self attention Pooling introduce extra operator like ATEN while trying to export on-device model to C2.
add Boundary Pooling in BiLSTMDocAttention to use only the last hidden state as output

Differential Revision: D13789842

fbshipit-source-id: 8fb749e79741720a422b9d5e7f8478068a7a92ad
gardenia22 added a commit to gardenia22/pytext-1 that referenced this pull request Jan 31, 2019
Summary:
Pull Request resolved: facebookresearch#256

Self attention Pooling introduce extra operator like ATEN while trying to export on-device model to C2.
add LastTimestep Pooling in BiLSTMDocAttention to use only the last hidden state as output

Differential Revision: D13789842

fbshipit-source-id: 3aedfafe7b90c987988f6d6fc426d4de679cac5e
gardenia22 added a commit to gardenia22/pytext-1 that referenced this pull request Jan 31, 2019
Summary:
Pull Request resolved: facebookresearch#256

Self attention Pooling introduce extra operator like ATEN while trying to export on-device model to C2.
add LastTimestep Pooling in BiLSTMDocAttention to use only the last hidden state as output

Differential Revision: D13789842

fbshipit-source-id: 3ba8dceba569f265badd8a38cf679f00f48cdbda
Summary:
Pull Request resolved: facebookresearch#256

Self attention Pooling introduce extra operator like ATEN while trying to export on-device model to C2.
add LastTimestep Pooling in BiLSTMDocAttention to use only the last hidden state as output

Differential Revision: D13789842

fbshipit-source-id: 69b5746846a6a4163dc9214db3cc6f8827832d5a
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants