Skip to content

MAINT Changed the use of ColumnTransformer to make_column_transformer #831

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
May 27, 2025

Conversation

SebastienMelo
Copy link
Contributor

Per issue #452, made the use of make_column_transformer more uniform in the MOOC. Since the name of the preprocessors isn't used to modify their parameters, it seems simpler to teach the students to create a ColumnTransformer with the function.

Copy link
Collaborator

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I will sync with main and regenerate the notebooks and then merge if all green.

@ogrisel ogrisel merged commit 6625d0c into INRIA:main May 27, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request May 27, 2025
…ransformer (#831)

* changed besides to additionally for better phrasing

* Apply suggestions from code review

* Changed the use of ColumnTransformer to make_column_transformer

* fixed format

* fixed format

* changed additional mentions of ColumnTransformer

* Rerender notebooks

---------

Co-authored-by: SebastienMelo <seastien.melo@polytechnique.edu>
Co-authored-by: Arturo Amor <86408019+ArturoAmorQ@users.noreply.github.com>
Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org> 6625d0c
ArturoAmorQ pushed a commit to ArturoAmorQ/scikit-learn-mooc that referenced this pull request Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants