ufal / neuralmonkey

An open-source tool for sequence learning in NLP built on TensorFlow.
BSD 3-Clause "New" or "Revised" License
410 stars 106 forks source link

max_length must be undefined when using SequenceLabeler #790

Open varisd opened 5 years ago

varisd commented 5 years ago

When using SequenceLabeler, the max_length argument must be unset for the underlying encoder (or input_sequence). Otherwise, there will be a shape mismatch in SequenceLabeler.cost for longer training sequences.

This is an ok behaviour, however, there is no indication of this requirement (warning, etc.) which can be quite confusing for the user.

jlibovicky commented 5 years ago

To už mám taky zafixovaný ve větvi.