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

speed up travis builds #125

Merged
merged 2 commits into from Jan 2, 2019
Merged

speed up travis builds #125

merged 2 commits into from Jan 2, 2019

Conversation

falkben
Copy link
Contributor

@falkben falkben commented Jan 2, 2019

this seems to speed up the travis builds by about 50%.

i believe the big improvement is from cacheing the packages

the suggestion from the following blog post is to use remotes instead of the r_packages entry we currently have in the travis.yml file, but I haven't tried that.

https://blog.rstudio.com/2016/03/09/r-on-travis-ci/

related to #123

@falkben falkben requested a review from MrAE January 2, 2019 19:51
Copy link
Collaborator

@MrAE MrAE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

@falkben falkben merged commit 7a691d6 into staging Jan 2, 2019
@falkben falkben deleted the travis_adjust branch January 2, 2019 21:01
MrAE added a commit that referenced this pull request Feb 6, 2019
* Add Zenodo DOI Badge. (#118) [Closes #74]

* Add Zenodo DOI Badge.

* Fix link [Closes #74]

* speed up travis builds (#125)

* removed the distribution and sudo entries from travis config - faster?

* adding back sduo false and adding cache packages option

* small updates to contributing guide (#133)

* add example for running styler

move contributing to .github folder

* ignore the .github path

* fix indexing order of operations error [fixes #119]. (#134)

* added functionality to change mtry and sparsity in Urerf (#120)

* added functionality to change mtry and sparsity in Urerf

* ran styler on modified files and removed white space.

* added tests for new RandMat functions.

* Added the functionality to splitting based on BIC score using Mclust (#124)

* added functionality to change mtry and sparsity in Urerf

* Added functionality to split based on BIC score

* Add LinearCombo arg to the Urerf fn

* Add fast version of BIC

* fix some minor errors (#141)

Ran through styler and fixed some roxygen import and documentation.

* fix issue #91 based on discussion in the comments. (#140)

* fix issue #91 based on discussion in the comments.
add some helper functions
add test for new way of computing feature importance

* remove need for library(Matrix) and update function parameteres.
fix documentation typos
[issue #91]

* update test-FeatureImportance
move `flipWeights` to helperFunctions

* update Feature Importance to be more readable [@ben].
Merge RunFeature* into the same file.
Update README with correct output names.

* check-as-cran warning will now cause TravisCI to fail. (#142)

* Print tree (#136)

* added functionality to change mtry and sparsity in Urerf

* ran styler on modified files and removed white space.

* added tests for new RandMat functions.

* added PrintTree function and modified NAMESPACE file to call PrintTree (I'm not sure this last step was necessary but it doesn't hurt.

* Add documentation and adjust the formatting of the output.

* the double comparison now relies on machine epsilon. (#149)

* the double comparison now relies on machine epsilon.

* fix for test not passing

* move an assignment out of an if condition. (#151)

Fixes issue #135

* Packed forest submodule (#152)

* add packedForest submodule

* update submodule to latest commitadd readme for submodule operations

* update submodule readme

* update submodule

* update submodule (#154)

* update submodule (#155)

* Draft of v2.0.3 for CRAN (#156)

* Draft of v2.0.3 for CRAN
no warnings, errors, or notes on my Mac.

* run README.Rmd

* update submodule (#159)
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

Successfully merging this pull request may close these issues.

None yet

2 participants