search

firefly-cpp / NiaARM

A minimalistic framework for Numerical Association Rule Mining
MIT License
15 stars 6 forks source link

Test NiaARM package on Windows #38

Closed firefly-cpp closed 2 years ago

firefly-cpp commented 2 years ago

NiaARM is working well on Linux instances.

@zStupan, Have you already tried to install/run NiaARM on Windows?

zStupan commented 2 years ago

I have not, maybe we can add it to the CI workflow. And macOS as well.

zStupan commented 2 years ago

The tests are failing on windows: https://github.com/zStupan/NiaARM/runs/5829344958?check_suite_focus=true

zStupan commented 2 years ago

It's the same issue as #27. I think pandas just doesn't work properly on 32 bit architectures or something.

UPDATE: The problem was that on 32bit platforms np.int64 is not an instance of int and vice versa, so it treated integer variables as categories. It is fixed in the latest PR.