INFO-526-S24 / project-final-PlotWizards

https://info-526-s24.github.io/project-final-PlotWizards/
0 stars 0 forks source link

Code peer review #4

Closed lakshmineharika closed 4 months ago

lakshmineharika commented 4 months ago

The following is the peer review of the project code by [datavista]. The team members that participated in this review are

Abhishek Kumar, MS in Data Science - @Abhi291712

Noureen Mithaigar, MS in information science - @musketer26

Akash Srinivasan, MS in Data Science - @AkashSrinivasan12

Lakshmi Neharika Anchula, MS in Data Science - @lakshmineharika

Gowtham Theeda, MS in Data Science - @gowthamtheeda

Divya Liladhar Dhole, MS in Data Science - @Divyadhole

1. Purpose of Code

The code is used to create a Shiny app that lets users explore cricket statistics interactively. By using this we can compare player and team performances over different seasons and see key partnerships among them through charts and tables. The code is clear and understandable. There is no data preprocessing steps,and labels to chunks


2. Code Quality

The code is clean, well-organized, and includes a sufficient number of comments. However The variables have meaningful names, and the overall code style is consistent, with appropriate naming conventions.

3. Documentation

code documentation is sufficient.


4. Best Practices

There is no hard coded variables,No such security concern is seen in the data,no repetition is seen as such.


5. Additional Comments

6. Recommendation

Approved with minor issue- can be addressed later.


NandhiniAnne commented 4 months ago

Thanks for your feedback!!!!