UBC-MDS / Abalone_Age_Prediction

MIT License
0 stars 8 forks source link

EDA Plots and Preprocessing #13

Closed charlessuresh closed 3 years ago

charlessuresh commented 3 years ago
chuangw6 commented 3 years ago

Hi @charlessuresh , For the code quality purpose, in the eda script, can you break down the main function into three helper functions where each function is meant for plotting one figure only? And then you can call these three helper functions in the main function, which will make the code much cleaner. Thanks

charlessuresh commented 3 years ago

Hey @chuangw46 That's a great suggestion. I've added helper functions- one for each of the three plots. Request you to review the amends made.