awsm-research / PyExplainer

PyExplainer: A Local Rule-Based Model-Agnostic Technique (Explainable AI)
MIT License
29 stars 9 forks source link

import RuleFit error #7

Closed oathaha closed 3 years ago

oathaha commented 3 years ago

I found that the below code from 'pyexplainer_pyexplainer.py' that import RuleFit triggers an import error from pyexplainer.rulefit import RuleFit

Here is the error message image

I found that using this code can solve the problem from rulefit import RuleFit

I found this error because I want to import pyexplainer_pyexplainer.py directly from pyexplainer directory, to ensure that I use the latest code for my experiment.

So my request is changing from pyexplainer.rulefit import RuleFit to from rulefit import RuleFit in master branch.

MichaelFu1998-create commented 3 years ago

this will not pass the pytest, let me do something else to fix this.

oathaha commented 3 years ago

I think if you want to use the original code to import RuleFit, you should update python package in pypi so I can use the latest version of pyExplainer via pip install ...

MichaelFu1998-create commented 3 years ago

The correct approach should be from .rulefit import RuleFit I've released a new version on PyPI which is 0.1.10 Thanks for pointing out this!