search

skrub-data / skrub

Prepping tables for machine learning
https://skrub-data.org/
BSD 3-Clause "New" or "Revised" License
1.16k stars 97 forks source link

FIX make sure to have Mixin on the left of BaseEstimator #931

Closed glemaitre closed 3 months ago

glemaitre commented 3 months ago

From the top of the head, we should always have the mixins on the left of BaseEstimator for the MRO to work properly (e.g. for tags inheritance).

Quick fix to solve issues that I let pass in a previous PR.