fire-ants / mlb-machine-learning

Test machine learning against MLB PitchFX data.
0 stars 0 forks source link

refactor hv_model to PEP8.. single-use functions #11

Closed goyetc closed 5 years ago

goyetc commented 5 years ago

hv_model(..) currently has several sub-scripts that perform different functions -import data -clean data -format for regression -perform regression -format results -print results -push to S3 Note this is not necessarily an inclusive/exhaustive list

We can have a cleaner script by improving/expanding upon functions defined within hv_model. Also can add additional arguments up front to reduce if statements within, and easy define whether we want to print results or simply push them to the s3 bucket without an explicit print to output.

goyetc commented 5 years ago

Done, for the most part.