UBC-MDS / Abalone_Age_Prediction

MIT License
0 stars 8 forks source link

EDA code quality #17

Open hsmohammed opened 3 years ago

hsmohammed commented 3 years ago
chuangw6 commented 3 years ago

Hi @hsmohammed, Thank you so much! great feedback. I didn’t pay attention to the duplicated code last week. I have made changes here and the repo is updated.

Thank you for sharing the resource. As we intend to draw distribution for each column, the faceted histogram might not work here. Therefore, I use the for loop to draw the histogram for each column. This reduces the duplicated code and follows the DRY principle. I hope this is okay.

Thank you again for your time.