-
Notifications
You must be signed in to change notification settings - Fork 45
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
lack of automated testing #33
Comments
To be fair, GLMsingle is not unique in its lack of testing but it does not mean we can't try to raise the bar. |
Yes, thanks for the nudge. We should make our way towards this. Let me confer with @iancharest. |
Remi, Ian and I have been working through the issues/bugs/etc. on the repo. Currently, we have run the MATLAB and Python implementations on a small test dataset, and can confirm that the two implementations give (nearly) identical results.... In terms of the larger issue of automated unit testing, we think the code is in a good state to pursue that. If you feel motivated to work on this, we are happy to review/discuss and would be grateful for such efforts. Let us know if you want to discuss. |
Yup happy to give suggestions and maybe help set some foundations if you are interested. Feel free to ping me at |
One of my concern regarding using GLMsingle is the lask of either unit or system test which creates an uncertainty whether the toolbox is giving expected results.
Tests can usually be easily implemented with external toolbox (moxunit for matlab) or packages (pytest for python).
Automating those tests (through github actions for example) can usually also prevent regression (breaking old feature A when implementing new feature B or fixing bug C).
If there is some interest I could help with setting some of these up.
The text was updated successfully, but these errors were encountered: