georgian-io-archive / foreshadow

An automatic machine learning system
https://foreshadow.readthedocs.io
Apache License 2.0
29 stars 2 forks source link

Clean up the properties in the foreshadow class #156

Open jzhang-gp opened 5 years ago

jzhang-gp commented 5 years ago

This might be the best time to clean up the properties that were used. I don't think the properties is the best way of auto initializing those attributes and think they should all be moved to the init, similar to what you just proposed :)

Originally posted by @cchoquette in https://github.com/georgianpartners/foreshadow/pull/154