sta199-s23-2 / project-r-s2dio

https://sta199-s23-2.github.io/project-r-s2dio/
0 stars 0 forks source link

Peer review #6

Open dmf7 opened 1 year ago

dmf7 commented 1 year ago

There appear to be several goals given that there are two research questions because of a merge conflict, but the main goal appears to be to explain how regular season efficiency correlates to post season and tournament success (shown by seeding)

The data was collected from a website that includes D1 college basketball stats from 2013-2019, and the dataset was found, not collected.

They used scatterplots to visualize the relationship between their chosen variables and three linear regression models to analyze their data. They mutated the dataset and did not use the mutated dataset?

They used adj.r.squared to analyze the linear models.

The statistical visualizations show a strong logic that gets after their research question. However, there could be more transparent reasoning in the form of a more concrete and thoughtful conclusion. They could also include a visualization that includes a depiction of all three variables, especially given that they conclude an interaction model is the best predictor. They could also add lines of best fit to their original models in order to better demonstrate the correlations.

They could stand to visualize more aspects of the model as well, specifically in relation to postseason success and offensive vs. defensive efficiency.

We are interested in the relationship between offensive and defensive efficiency—from our deductions it seems that high seeds have high offensive and low defensive efficiency, which piques our curiosity.

No, the Render did not work because the data was not loaded in a way that users could access it. For example, there was no read.csv command that would allow users to view the data.

There is no project title; there are some unresolved merge conflicts; the question prompts from the guidelines are still in the report; the code chunks should be labeled and graphs should have titles.

The idea of comparing the success of numerous methods of modeling the same variables is very interesting, and we will definitely take that into our project!

angeliequimbo commented 1 year ago

Fixed the render by adding read.csv. We will update the workflow discrepancies (labelings and merge conflicts)