Skip to content

Navigation Menu

Sign in

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

DataFrameMapper using FeatureUnion #64

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
Loading
from
Open

DataFrameMapper using FeatureUnion #64

wants to merge 4 commits into from

Conversation

dukebody
Copy link
Collaborator

This PR tries to solve #61 and builds on #62 with the following important additions:

  • get_params and set_params methods to be able to modify estimator parameters during grid-search and others.
  • improved naming for steps, using sklearn's _name_estimators.
  • Allow to specify custom feature names in the third element of the feature definition tuple to avoid (I believe unfrequent) name collisions.

Perhaps we are missing some extra documentation in README.rst?

@dukebody dukebody mentioned this pull request Sep 3, 2016
Open
@cstjean cstjean mentioned this pull request Feb 8, 2017
Open
@vsriram11 vsriram11 mentioned this pull request Apr 28, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dukebody
Morty Proxy This is a proxified and sanitized view of the page, visit original site.