search

kristinawright / group_09-1

Repository for Kristina Wright and Daniel Hadley group project
MIT License
0 stars 0 forks source link

Peer Review #7

Closed kristinawright closed 4 years ago

kristinawright commented 4 years ago

This is referencing qiyangqd's issue

Hi @qiyangqd,

Thank you for your feedback! My partner and I have disbanded, so you will see these issues addressed twice.

I have applied the following feedback:

The unused test scripts .testieTest.Rmd and Hadley-Test.Rmd have been removed.

The Euro symbol was replaced with the unicode u20AC, which I think will work on all systems.

The usage script names are all updated and include the script directory in the path name.

The following feedback is for future work:

I will be working on vectorizing the mean price for the violin_plot and the Issue for this future work is added here

Thanks again! Kristina