-
-
Notifications
You must be signed in to change notification settings - Fork 26.9k
discrete branch: add encoding option to discretizer #9336
Copy link
Copy link
Closed
Labels
EasyWell-defined and straightforward way to resolveWell-defined and straightforward way to resolveEnhancementhelp wanted
Description
We recently merged a discretizing transformer into the discrete branch (see diff between that branch and master). Before merging it into master, we'd like it to be able to one-hot encode the output by default.
I propose that we have a parameter encode='onehot' by default which produces a one-hot encoded sparse matrix, with other options 'ordinal' (for the current default output) and encode-dense which produces a dense version of 'onehot'.
To dear contributor: Make sure to submit a pull request to the discrete branch.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
EasyWell-defined and straightforward way to resolveWell-defined and straightforward way to resolveEnhancementhelp wanted