[URGENT] Wrangling script does not run

UBC-MDS / DSCI_522_Group-308_Used-Cars

This project attempts to build a regression model to predict price of used cars based on numerous features of the car

MIT License

2 stars 6 forks source link

Closed ksedivyhaley closed 4 years ago

ksedivyhaley commented 4 years ago

Group308_Error2

ksedivyhaley commented 4 years ago

Also fix:

Wrangling script removes outliers before splitting into test/train - splitting should be your first step.
Script initial summary doesn't describe target argument (but is described under options) - otherwise well documented

Additional feedback:

Could use prefix to simplify train/test path arguments
Is it a good idea to use 0.9 as your default train size, given that this is only appropriate for larger datasets?
Good use of tests in multiple sub-functions.

AndresPitta commented 4 years ago

Fixed, Thanks Kate!