ufal / neuralmonkey

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

Objective.name format should be more strict #787

Open varisd opened 5 years ago

varisd commented 5 years ago

The Objective.name in general should have more strict naming format, otherwise it cannot be used e.g. for naming graph parts (for example naming Delayed trainer accumulators with corresponding objective names for better debugging)

https://github.com/ufal/neuralmonkey/blob/master/neuralmonkey/trainers/objective.py#L85

At least whitespaces should not be allowed