UBC-MDS / fifa-potential

Supervised machine learning model to predict potential rating of players in FIFA 22
https://ubc-mds.github.io/fifa-potential/high-potential-fifa-prediction-report.html
Other
0 stars 0 forks source link

Edits to make based on Milestone 1 feedback #36

Closed jbarns14 closed 7 months ago

jbarns14 commented 7 months ago
srfrew commented 7 months ago

Just a heads up that I'm working on updating the EDA scripts to be black formatting compliant in the bash-makefile branch. Once that's merged in can we split off to update EDA plots as recommended in milestone 1? Thoughts?

jbarns14 commented 7 months ago

Sure that sounds good. Is it not just a matter of updating the eda plot function script to put legends on the plots?

srfrew commented 7 months ago

Ah yeah you're right. It doesn't actually need to wait till I finish what I'm doing

meretelutz commented 7 months ago

Version tags were added before the milestone 2 submission!

jbarns14 commented 7 months ago

When it comes to adding descriptions to the plots, that will have to be done once we get the notebook file rendering only the plots we want the reader to see. Is there any chance whoever sets up the notebook rendering can add the plot descriptions? It should only need to be one maybe 2 sentences max just describing the plot.

meretelutz commented 7 months ago

The plot actually already has a legend on the side, it must've just been cutoff whenever the grader was looking at the report. Maybe we need to just tweak the sizing of the plots instead so it all fits on a single page

srfrew commented 7 months ago

Closing as updates have been merged into main and reference-bibtex and report file