{"push_id":686834666,"size":5,"distinct_size":5,"ref":"refs/heads/master","head":"db29e5d79912cf7f20783c81897876658352d73c","before":"954810560a0982636f94c6de502d30fd9748b2e2","commits":[{"sha":"38d73438fd74625611d4212f7b51c28a1919c286","author":{"email":"b57e82594a2b82d0d45df1b8a5713a1613d1ce16@gmail.com","name":"Tiffany Timbers"},"message":"Merge pull request #4 from QMuncaster/master\n\nsize and pathlength violin plots - updated (May 19)","distinct":true,"url":"https://api.github.com/repos/QMuncaster/MWT_Locomotion_summary_analysis/commits/38d73438fd74625611d4212f7b51c28a1919c286"},{"sha":"9c060db758afcab1ad70a5608e54d043065aab70","author":{"email":"aeb8b9870cf99471ed667d46946eeaa9a7bdf9cb@sfu.ca","name":"Kwangjin1986"},"message":"wrote some comments","distinct":true,"url":"https://api.github.com/repos/QMuncaster/MWT_Locomotion_summary_analysis/commits/9c060db758afcab1ad70a5608e54d043065aab70"},{"sha":"4fb3303609d27ab09f780957a6a7e84033aed56b","author":{"email":"b57e82594a2b82d0d45df1b8a5713a1613d1ce16@gmail.com","name":"ttimbers"},"message":"Fixed chore.jar issue and most of the R script\n\nChore.jar issue:  apparently in the --output call, C refers to a plugin? That is the issue,\nif you remove the C and keep all other output calls in there, it works. Don't know why,\ndon't care.\n\nR script issues:\n\n1. When writing a function, you also need to call that function, or it doesn't happen (this\nwas an issue for the plot.speed.time() function. It now works.\n\n2. When using ggplot() we can use ggsave() instead of pdf(). In doing so, we don't have\nto do dev.off() and we have more options of how big to make the figures, lets use this\nin the future.\n\n3. let's load our libraries using require() instead of library(), and let's do this in\nthe main function at the beginning of the script instead of within the functions (e.g.\nmost of our functions require the ggplot2 library, so why not call it once in the main\nscript instead of in each function).\n\n4. Watch the names of variables in the functions, I had to fix several of these.\n\n5. I made it so that all the .pdfs are getting saved to the results directory.\n\n6. There is still an error in one of the functions dealing with pathlength. I get this\nerror:\n\n$ rscript bin/Column_identification.R data/chore_data/merged.file\n\nLoading required package: ggplot2\nLoading required package: methods\nLoading required package: plyr\nError in `[<-.data.frame`(`*tmp*`, , \"pathlength\", value = c(6.52181241733473,  :\n  replacement has 162 rows, data has 174\nCalls: main -> mean.pathlength -> [<- -> [<-.data.frame\nExecution halted","distinct":true,"url":"https://api.github.com/repos/QMuncaster/MWT_Locomotion_summary_analysis/commits/4fb3303609d27ab09f780957a6a7e84033aed56b"},{"sha":"4d43453c657e446fe470d095d796b2d97879cb95","author":{"email":"b57e82594a2b82d0d45df1b8a5713a1613d1ce16@gmail.com","name":"ttimbers"},"message":"Added some data","distinct":true,"url":"https://api.github.com/repos/QMuncaster/MWT_Locomotion_summary_analysis/commits/4d43453c657e446fe470d095d796b2d97879cb95"},{"sha":"db29e5d79912cf7f20783c81897876658352d73c","author":{"email":"e4067f55323b4b17db5271cde912aef6a12acf33@gmail.com","name":"Quintin Muncaster"},"message":"Merge remote-tracking branch 'ttimbers/master'\n\nConflicts:\n\tbin/Column_identification.R","distinct":true,"url":"https://api.github.com/repos/QMuncaster/MWT_Locomotion_summary_analysis/commits/db29e5d79912cf7f20783c81897876658352d73c"}]}