Skip to content

Commit

Permalink
[ENH] Enable Grid-Search for TableVectorizer (skrub-data#814)
Browse files Browse the repository at this point in the history
* apply global sub-estimator default parameter

* fix docstirng

* fix docstring

* remove pytest exception

* add changes

* Update skrub/_table_vectorizer.py

Co-authored-by: Jérôme Dockès <[email protected]>

* remove the None conversion to passthrough

* add passthrough as default for numerical_transformer

* fix precommit

---------

Co-authored-by: Jérôme Dockès <[email protected]>
  • Loading branch information
Vincent-Maladiere and jeromedockes authored Nov 9, 2023
1 parent 2bda119 commit 4b11e62
Show file tree
Hide file tree
Showing 4 changed files with 208 additions and 241 deletions.
4 changes: 4 additions & 0 deletions CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ development and backward compatibility is not ensured.
Major changes
-------------

* Pipelines including :class:`TableVectorizer` can now be grid-searched, since
we can now call `set_params` on the default transformers of :class:`TableVectorizer`.
:pr:`814` by :user:`Vincent Maladiere <Vincent-Maladiere>`

* :func:`to_datetime` is now available to support pandas.to_datetime
over dataframes and 2d arrays.
:pr:`784` by :user:`Vincent Maladiere <Vincent-Maladiere>`
Expand Down
3 changes: 1 addition & 2 deletions skrub/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
from ._minhash_encoder import MinHashEncoder
from ._select_cols import DropCols, SelectCols
from ._similarity_encoder import SimilarityEncoder
from ._table_vectorizer import SuperVectorizer, TableVectorizer
from ._table_vectorizer import TableVectorizer
from ._target_encoder import TargetEncoder

check_dependencies()
Expand All @@ -29,7 +29,6 @@
"GapEncoder",
"MinHashEncoder",
"SimilarityEncoder",
"SuperVectorizer",
"TableVectorizer",
"TargetEncoder",
"deduplicate",
Expand Down
Loading

0 comments on commit 4b11e62

Please sign in to comment.