Add rule induction algorithms - Githubissues

Trusted-AI / AIX360

Interpretability and explainability of data and machine learning models

https://aix360.res.ibm.com/

Apache License 2.0

1.63k stars 307 forks source link

Add rule induction algorithms #148

Closed kmyusk closed 2 years ago

kmyusk commented 2 years ago

Includes the Ripper algorithm and TRXF ruleset exchange format.

The rule_induction directory will eventually contain a set of closely related algorithms (drop-in replacements) that are used to induce and export rule set in the common TRXF format for consumption by AIMEE, ADS, and RedHat Decision Manager, etc. This originates from the internal aix360i:ripper branch, and we intend to migrate parts of that code as the quality bar is achieved.

In particular, as previously discussed with @vijay-arya, the reason for this migration to the public repo is to provide the more technical ADS (CP4BA) clients with the means to programmatically generate their own rule sets without relying on the AIMEE GUI.

Required dependencies:

numpy
pandas
sklearn
nyoka
xmltodict
numba

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 8c2f9e1eb692c77136c02f3ea4ee99c1170d695b into e06d186eaa67a9f42b183372933e9bbad64d8fd5 - view on LGTM.com

new alerts:

1 for Conflicting attributes in base classes
1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 2d471ff25a9ea0128aa532abfc3cbaeb213ddedd into e06d186eaa67a9f42b183372933e9bbad64d8fd5 - view on LGTM.com

new alerts:

1 for Conflicting attributes in base classes
1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging f3e9beaed1bdc6faff8967d5b126c1fb517406fb into 0a6ab2c897e15345f1673024f9b765c82ef778a5 - view on LGTM.com

new alerts:

1 for Unused import

kmyusk commented 2 years ago

There are some duplicate commits after rebase/merge conflict resolution. DCO failed because the earlier commit 635e009 "Added ripper demo notebook" was not signed-off, but the later one 31f01e0 was.

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging f0315f899cc90a2e18604648f2778d7618a44426 into 0a6ab2c897e15345f1673024f9b765c82ef778a5 - view on LGTM.com

new alerts:

1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 4c70d4f7f5215955859e7b1b087b0332310d6f3c into 0a6ab2c897e15345f1673024f9b765c82ef778a5 - view on LGTM.com

new alerts:

1 for Unused import

vijay-arya commented 2 years ago

@kmyusk your PR is merged. Thanks! for integrating rule induction into aix360.