search

rwth-i6 / returnn

The RWTH extensible training framework for universal recurrent neural networks
http://returnn.readthedocs.io/
Other
348 stars 130 forks source link

Switch to PyTest #987

Closed albertz closed 2 years ago

albertz commented 2 years ago

Currently we use nosetests.

I think PyTest has become much more popular.

I'm also not exactly happy with the error reporting of nosetests. E.g. I would like some nice summary at the end which lists the failing test cases. Not sure if PyTest has that though.

But in any case, it is often a good idea to use whatever is more widely used, as this is better supported. And this should probably be simple to switch.

albertz commented 2 years ago

Any opinions? @curufinwe @patrick-wilken @Zettelkasten ?

patrick-wilken commented 2 years ago

I wouldn't mind switching, but also doesn't sound like a top priority. 😛

JackTemaki commented 2 years ago

I missed this one, yeah I also saw that nosetests is no longer maintained and people should switch to something else. So I guess switching to PyTest is not a bad idea.