-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add Apriori algorithm also for better comparison #88
Comments
Thanks, @minakshikaushik. This is a good point. Although NiaARM is more or less oriented for numerical association rule mining, having a referential Apriori implementation in this repository would be beneficial for comparison, as you mentioned. @zStupan, what do you think? Are you interested in adding basic Apriori implementation? Employing some classes from NiaARM may help speed up the performance. The discretization step exists in the tinynarm library. By the way, there is a lack of pure Python Apriori implementations equipped with good documentation and still under active maintenance. |
@zStupan, what do you think? |
Hello, sorry for the late response, I don't really have a lot of free time right now... We could add an implementation of Apriori, although there are already some popular implementations out there (efficient-apriori). So if I understand correctly, the comparison would be our method vs discretization and then running apriori? |
@minakshikaushik, what do you think/need? A comparison that may be suitable is our method using discrete dataset vs. apriori in terms of the number of rules generated, supports, confidences and time (very important), etc. @zStupan, do you think including any existing implementations may be more convenient? |
@firefly-cpp |
@zStupan, any progress, maybe? |
No description provided.
The text was updated successfully, but these errors were encountered: