search

smazzanti / mrmr

mRMR (minimum-Redundancy-Maximum-Relevance) for automatic feature selection at scale.
MIT License
531 stars 79 forks source link

fix: add missing polars requirement #27

Closed jose-turintech closed 1 year ago

jose-turintech commented 1 year ago

add missing polars requirement as it is needed since: https://github.com/smazzanti/mrmr/commit/be8479a9544dc60a48778dfb78a5a48f9c3445f5 where polars support was included.

If polars is not meant to be used by default, further module management and maybe extra support for polars would be good to have.

roesgaard commented 1 year ago

I have created a new PR, please see if this resolves your issue

fix: add missing polars requirement 2