Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CLI + Added a simpler interface for mining association rules #29

Merged
merged 8 commits into from
Mar 11, 2022

Conversation

zStupan
Copy link
Collaborator

@zStupan zStupan commented Mar 11, 2022

  • Updated CLI for the current interface (added metrics and weights flags)
  • Added a --version flag that prints out the current version of the library
  • Metrics: Yule's Q, netconf and conviction would throw division by zero errors, for rules with support 1. There are a lot of rules generated that are true for every transaction (the entire ranges of features would be selected). I just added a small value to the denominators to avoid this.
  • Added the get_rules function which provides a simpler interface for mining rules, it.
  • Added a RuleList class, which is a wrapper around a regular list of rules but you can get stats, sort rules by metric and export to csv. It replaces the stats class, and the rule sorting and export functionalities are moved to it from NiaARM
  • closes Measure running time of an algorithm #26, since get_rules also returns run time.

@zStupan zStupan changed the title Update CLI Update CLI + Added a simpler interface for mining association rules Mar 11, 2022
@firefly-cpp
Copy link
Owner

Excellent work, @zStupan!

@firefly-cpp firefly-cpp merged commit a73919f into firefly-cpp:main Mar 11, 2022
@zStupan zStupan deleted the update-cli branch March 11, 2022 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Measure running time of an algorithm
2 participants