Skip to content
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

downsized data sets to speed up testing #26

Open
abigailsnyder opened this issue Mar 22, 2019 · 1 comment
Open

downsized data sets to speed up testing #26

abigailsnyder opened this issue Mar 22, 2019 · 1 comment

Comments

@abigailsnyder
Copy link
Contributor

Currently devtools::test() takes a long time because our included sample data is large. At some point, we could figure out smaller sample data that works for the tests just as well.

@rplzzz
Copy link
Contributor

rplzzz commented Apr 12, 2019

The length of the test suite is a problem, but it looks like many of our travis timeouts (mostly on OS X, since the travis OS X instances are slow) are being caused by building the vignettes. I wonder if we could mitigate some of this by eliminating the tutorial1 vignette, which seems to be mostly obsolete.

The other question is, can we speed up the vignette processing? Reducing the resolution of the example data might help. If we halve the resolution in longitude, leaving latitude alone, we halve the size of a bunch of matrices, which could dramatically reduce the time required for some of the SVD and matrix multiplies dramatically (as I read it, for an m x n matrix, SVD is O(m^2*n + n^3) -- with n as the long dimension, that's effectively n^3).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants