Open bcahn7 opened 6 years ago
Peer Review:
Report:
summarize_each()
function.gsub()
is just an alternative to str_replace()
?ret_func()
. It looks to me if you just substitute x with mkt_data and use all the pipelines defined in this function then it'll work.Scripts and makefile:
all
, I think you can just put 03_report.md
because the other .png
plots are included in the report file.*.png
. I should use this too:)Hi Its great to see someone using some other data here and its a really great homework.
Few comments:
Overall with little bit effort I was able to understand your work without having any background so its clear you explained everything very well and piped everything very efficiently so overall its great work, its a very well done homework (keep it up!!!)
Cheers Vibudh
Hi @bcahn7 ! Here are some comments about your hw07:
Three or more scripts, an Rmd, and a Makefile: Yes Starts by downloading data, ends with Rmd: Yes The output of each is the modified input of the previous step: Yes Includes some analysis and at least one figure: Yes Makefile includes all scripts and Rmd with correct dependencies: No (see comments) Makefile runs: Yes
Note: your mark (check minus, check, check plus) will be distributed later.
hw07 ready for grading