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 mm_model_by_ply, other refinements #180

Merged
merged 8 commits into from
Mar 25, 2016

Conversation

aappling-usgs
Copy link
Contributor

also added a 'modal' option to mm_get_timestep. big thing is the speed up, though

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 60.948% when pulling 1d5c71e on aappling-usgs:develop into 687320a on USGS-R:develop.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 60.948% when pulling ace805d on aappling-usgs:develop into 687320a on USGS-R:develop.

@aappling-usgs
Copy link
Contributor Author

current AppVeyor issues in build 1.0.233, line 287:

Installing GitHub packages: GLEON/LakeMetabolizer
+ Rscript -e 'library(devtools); library(methods); options(repos=c('\''http://cran.rstudio.com'\'','\''http://owi.usgs.gov/R'\'')); install_github(commandArgs(TRUE), build_vignettes = FALSE)' GLEON/LakeMetabolizer
Downloading GitHub repo GLEON/LakeMetabolizer@master
from URL https://api.github.com/repos/GLEON/LakeMetabolizer/zipball/master
Installing LakeMetabolizer
Skipping 2 unavailable packages: plyr, rLakeAnalyzer
"c:/R/bin/i386/R" --no-site-file --no-environ --no-save --no-restore CMD  \
  INSTALL  \
  "C:/Users/appveyor/AppData/Local/Temp/1/RtmpGyKHgz/devtools4384ff7129e/GLEON-LakeMetabolizer-ad6f501"  \
  --library="c:/RLibrary" --install-tests 

ERROR: dependencies 'rLakeAnalyzer', 'plyr' are not available for package 'LakeMetabolizer'
* removing 'c:/RLibrary/LakeMetabolizer'
Error: Command failed (1)
Execution halted
Command exited with code 1

@aappling-usgs
Copy link
Contributor Author

mostly made that new commit to kick of a new appveyor build now that i've updated USGS-R/r-appveyor and USGS-R/r-travis. also good to send travis to the new CRAN repo, i hope

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 60.948% when pulling 259d5d2 on aappling-usgs:develop into 687320a on USGS-R:develop.

@aappling-usgs
Copy link
Contributor Author

huzzah!

@aappling-usgs aappling-usgs merged commit 89306b4 into DOI-USGS:develop Mar 25, 2016
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