Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Commit

Permalink
correct the wrong parameter note(target_namespace) (#2987)
Browse files Browse the repository at this point in the history
Should be "tokens" rather than "target tokens"
  • Loading branch information
wlhgtc authored and DeNeutoy committed Jun 21, 2019
1 parent cf247c6 commit b6a2abb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion allennlp/models/encoder_decoders/simple_seq2seq.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ class SimpleSeq2Seq(Model):
The encoder of the "encoder/decoder" model
max_decoding_steps : ``int``
Maximum length of decoded sequences.
target_namespace : ``str``, optional (default = 'target_tokens')
target_namespace : ``str``, optional (default = 'tokens')
If the target side vocabulary is different from the source side's, you need to specify the
target's namespace here. If not, we'll assume it is "tokens", which is also the default
choice for the source side, and this might cause them to share vocabularies.
Expand Down

0 comments on commit b6a2abb

Please sign in to comment.