-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[WIP] Feature names with pandas or xarray data structures #16772
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
Closed
thomasjpfan
wants to merge
46
commits into
scikit-learn:main
from
thomasjpfan:feature_names_in_custom_array_out
Closed
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
37bf69f
TST Check
thomasjpfan 7599089
Merge remote-tracking branch 'upstream/master'
thomasjpfan 53e0260
Merge remote-tracking branch 'upstream/master'
thomasjpfan 60c84f5
Merge remote-tracking branch 'upstream/master'
thomasjpfan 9940a7d
Merge remote-tracking branch 'upstream/master'
thomasjpfan 156ec25
ENH Adds array_out
thomasjpfan cabb7c1
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan 6435391
STY Flake8
thomasjpfan edebf84
REV
thomasjpfan 496cf93
API crazy api changes lol
thomasjpfan ef30659
WIP More internal API changes
thomasjpfan 2071253
BUG
thomasjpfan 1c6b3d4
More streamline api (i hope)
thomasjpfan 2ef6815
DOC Add comment
thomasjpfan 95069e1
API More API thoughts
thomasjpfan e42333d
API Fix
thomasjpfan 49a3c34
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan c8e8e0b
ENH Copy for ndarray
thomasjpfan 2fd6300
ENH Better happening for sparse in xarray
thomasjpfan b23a2f3
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan 7ff1639
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan b6fbc51
BUG Fix test
thomasjpfan 7336cfe
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan 319ce56
CLN Simplifies array wrapping and unwrapping
thomasjpfan 71b13c8
CLN Rename custom class
thomasjpfan ffdd983
Bug Fix issues from renaming
thomasjpfan 621e6f4
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan c106856
ENH Do not crash for array-like
thomasjpfan 7c61307
ENH Everything is a duck
thomasjpfan f28190b
CLN Make sures the ducks quack
thomasjpfan c901ce8
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan 6e57487
WIP: Improves interface for array_out
thomasjpfan 7dc4338
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan 6a5b42a
STY Linting
thomasjpfan d97d6e6
WIP Adds more tests
thomasjpfan f0946f0
WIP Enables array_out for all transformers
thomasjpfan 34e74c3
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan cce6f42
ENH Adds get feature names out to imputers
thomasjpfan d434122
STY Lint fixes
thomasjpfan 3661f5a
STY Lint fixes
thomasjpfan 6d960a5
ENH Slightly better improvements
thomasjpfan 8b269a8
ENH Major refactor to QuantileTransformer
thomasjpfan 0348059
FIX Fixes get feature out names
thomasjpfan f70e7cd
ENH Adds feature names out for FeatureUnion
thomasjpfan 55f6b4f
MNT Fixes functiontransformer
thomasjpfan 4ee8f44
Merge remote-tracking branch 'upstream/master' into feature_names_in_…
thomasjpfan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this be
ndarray
instead ofdefault
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sometimes the output is sparse. The default means "sparse or ndarray"