search

rwth-i6 / returnn_common

Common building blocks for RETURNN configs, such as models, training concepts, etc
7 stars 4 forks source link

nn.choice default length_normalization unexpected #214

Open albertz opened 2 years ago

albertz commented 2 years ago

We have length_normalization=True by default. I think this is maybe unexpected.

length_normalization=False might be a better default. Or having it mandatory.

We can make it mandatory just on returnn_common side.

Otherwise, in RETURNN, this requires a new behavior version.