search

alteryx / compose

A machine learning tool for automated prediction engineering. It allows you to easily structure prediction problems and generate labels for supervised learning.
https://compose.alteryx.com
BSD 3-Clause "New" or "Revised" License
497 stars 46 forks source link

Drop python 3.7 support #299

Closed thehomebrewnerd closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #299 (95de4aa) into main (d8cda66) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #299   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           28        28           
  Lines         1330      1330           
=========================================
  Hits          1330      1330
thehomebrewnerd commented 2 years ago

assuming tests pass

Bad assumption

gsheni commented 2 years ago

What if we require pyarrow for Compose (or maybe just the unit tests)?

thehomebrewnerd commented 2 years ago

What if we require pyarrow for Compose (or maybe just the unit tests)?

I think it might be something else - I found found some posts indicating that upgrading to numpy 1.22 could fix this error, but I think we are constrained from doing this by other dependency requirements.