ClimbsRocks / auto_ml

[UNMAINTAINED] Automated machine learning for analytics & production
http://auto-ml.readthedocs.io
MIT License
1.64k stars 310 forks source link

fixed bug #417 #418

Open mohataher opened 5 years ago

mohataher commented 5 years ago

So evolutionary_search library did not pass their test cases.

On cv.EvolutionaryAlgorithmSearchCV class, it extends sklearn.model_selection._search.BaseSearchCV which requires _run_search method to be implemented. This method is not implemented on the library and therefore, it threw the error

TypeError: Can't instantiate abstract class EvolutionaryAlgorithmSearchCV with abstract methods _run_search

According to the implementation here at sklearn_deep, the method was ignored.

Therefore, I created a class that inherits EvolutionaryAlgorithmSearchCV and ignored that method.

coveralls commented 5 years ago

Coverage Status

Coverage decreased (-10.4%) to 69.767% when pulling d75e0c2bbc1c6fbe377032647c794579c1c960db on mohataher:master into 4b7e0e759023cdbc96b471c64bfd67203e2b5734 on ClimbsRocks:master.

mohataher commented 5 years ago

The overall coverage is seriously dropped. I'm happy to write test cases to get that back up. But I'm concerned this library is no longer maintained since @ClimbsRocks hasn't responded here or anywhere else in the past few weeks/months.

If you are still actively maintaining this, please let me know.

mohataher commented 5 years ago

@chjq201410695 have you pulled my changes on this fork? https://github.com/mohataher/auto_ml