Skip to content

Commit

Permalink
Minor fix: Lengths argument was not passed correctly in MultiHeadAtte…
Browse files Browse the repository at this point in the history
…ntion. (awslabs#282)

This is not a big deal as currently in the code the masking is only done
through the bias and not the lengths argument.
  • Loading branch information
tdomhan authored and fhieber committed Jan 29, 2018
1 parent 073f88c commit dc12c35
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion sockeye/layers.py
Original file line number Diff line number Diff line change
Expand Up @@ -513,7 +513,8 @@ def __call__(self,
return self._attend(queries,
keys,
values,
bias=bias)
bias=bias,
lengths=memory_lengths)


class ProjectedDotAttention:
Expand Down

0 comments on commit dc12c35

Please sign in to comment.