serengil / chefboost

A Lightweight Decision Tree Framework supporting regular algorithms: ID3, C4.5, CART, CHAID and Regression Trees; some advanced techniques: Gradient Boosting, Random Forest and Adaboost w/categorical features support for Python
https://www.youtube.com/watch?v=Z93qE5eb6eg&list=PLsS_1RYmYQQHp_xZObt76dpacY543GrJD&index=3
MIT License
456 stars 101 forks source link

Paralellism default value is True #22

Closed anapaulamendes closed 2 years ago

anapaulamendes commented 2 years ago

In the documentation it says that the default value to allow parallelism is False, but checking the code and commit history the default value is True.

It is necessary to adjust the documentation to be consistent with the code.

Code: https://github.com/serengil/chefboost/blob/5e1477af150d1dd16f9bc5e4a46acb3dc5f63ea8/chefboost/commons/functions.py#L85 README: https://github.com/serengil/chefboost/tree/5e1477af150d1dd16f9bc5e4a46acb3dc5f63ea8#paralellism

serengil commented 2 years ago

Merged the pull request. Thank you for your contribution.