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

Frozen the version of mypy and sacrebleu to fix failing tests in master branch #818

Closed varisd closed 5 years ago

jindrahelcl commented 5 years ago

já si nemyslim, že by se měla mrazit verze mypy. Když nová verze mypy najde nový chyby, znamená to, že už tam byly, a tim pádem je správně, že to začne padat. Když se zmrazí mypy, tak tam ty starý chyby zůstanou a nikdo mypy už nikdy neaktualizuje, protože se tam bude kumulovat backlog těch chyb.

varisd commented 5 years ago

Co ja vim. Kazdopadne ted neprochazi mypy ani v masteru a je potreba to resit. Nadava to na ta nova "generika" a nemel jsem cas se v tom vrtat (nikdy jsem to v minulosti nestudoval).

jindrahelcl commented 5 years ago

819

jindrahelcl commented 5 years ago

nejnovejsi verze sacrebleu je kompatibilni s mastrem, byl to jejich bug, takže bych to nehrotil.

jindrahelcl commented 5 years ago

Fixed in #819