georgian-io-archive / foreshadow

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

Fixing get params. #141

Closed cchoquette closed 5 years ago

cchoquette commented 5 years ago

Description

Fixes issue #129

Three major changes:

  1. switch to overriding _get_param_names for a cleaner way to get parent class params.
  2. switch PreparerStep to traverse mro to get all parent params
  3. Check and provide tests for major class get_params.