skrub-data / skrub

Prepping tables for machine learning

Home Page:https://skrub-data.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Consider casting to float32 by default in TableVectorizer

GaelVaroquaux opened this issue · comments

Problem Description

Using float64 instead of float32 typically incurs compute and memory loads, and users do not have this in mind.

Feature Description

We should add an option to the TableVectorizer to output float32. We should consider whether this is the default.

Alternative Solutions

N/A

Additional Context

N/A

that also applies (maybe even more) to encoders, for example MinHash outputs float64

Closing because this has been addressed in the postprocessing step of the TableVectorizer in #902