search

christophM / rulefit

Python implementation of the rulefit algorithm
MIT License
411 stars 112 forks source link

Argument description in rulefit.py needs corrected #40

Open caseywhorton opened 3 years ago

caseywhorton commented 3 years ago

Hello! The description of exclude_zero_coef on line 557 in rulefit.py states that True is the default, but False looks to be the default for the argument in the function:

exclude_zero_coef: If True (default), returns only the rules with an estimated coefficient not equalt to zero.

https://github.com/christophM/rulefit/blob/b1657af4b41df59e2ae64bb1767dbaf5ff1ed7fe/rulefit/rulefit.py#L557