You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Good idea. Another option would be to skip all the factorization tests from @Almond-S and let the boosting routines in general_tests only run for one iteration via control = boost_control(mstop = 1). Since we do not do any type checking in this file, I guess we could go with tests that simply check for no error
Activity
m-muecke commentedon Mar 19, 2025
You open for using GitHub Actions? Then I could provide a PR.
davidruegamer commentedon Mar 19, 2025
Yes and thank you! That would be awesome!
m-muecke commentedon Mar 20, 2025
Ok, great! You open for using testthat for testing? It's currently already in suggest, but afaik not really used.
davidruegamer commentedon Mar 20, 2025
IIRC, we didn't use testthat because our tests take a long time. I would have to think about how to simplify them to avoid long CI waiting times.
m-muecke commentedon Mar 20, 2025
What I generally do, when the tests are running too long for the CI is to add a skip call, e.g.
skip_on_ci()andskip_on_cran()davidruegamer commentedon Mar 20, 2025
Good idea. Another option would be to skip all the factorization tests from @Almond-S and let the boosting routines in general_tests only run for one iteration via
control = boost_control(mstop = 1). Since we do not do any type checking in this file, I guess we could go with tests that simply check for no error