-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Suggested library: busted #1
Comments
thank you - will review. agree that best to have a standard unit test library included |
btw Torch comes with a test framework too. See https://github.com/dibyendumajumdar/ravi-torch7/blob/master/doc/tester.md |
Well, torch do seem to overlap with a lot of internal tools. Probably not the time to wonder if that is a good thing, but, wasn't there a warning that torch is being "replaced" by pyTorch? Should this distro rely so much in a soon-to-be defunct project? Not questioning torch greatness, of course. Just ... wondering. |
Yes Torch is sadly now under maintenance. But that is okay as I expect to maintain and support my fork of Torch long term. Re the tools that overlap - that in itself is not an issue - I guess I have to review carefully whether the tool is used by enough projects. To be honest it may be that none of the libraries I am including use Busted (I haven't checked). |
If you plan to keep your torch fork, long-term, maybe, shifting from non-core "under maintenance" libraries would be the way to go. In the long-run that would be less code to maintain. Just a thought. |
Regarding busted usage, I've found that the libraries listed in the readme do not use busted. Some fellow suggested busted usage for luasocket (and that's about it): lunarmodules/luasocket#227 |
Although it would be nice to migrate stuff in torch it is less of an issue, I guess easier option is to add the most commonly used unit testing framework |
Testing library for lua: https://olivinelabs.com/busted/
The text was updated successfully, but these errors were encountered: