Skip to content

Fix Feature Request: Gaussian adjust option in RobustScaler #10139#10140

Closed
mabelvj wants to merge 1 commit intoscikit-learn:masterfrom
mabelvj:RobustScaler_gauss_adjust
Closed

Fix Feature Request: Gaussian adjust option in RobustScaler #10139#10140
mabelvj wants to merge 1 commit intoscikit-learn:masterfrom
mabelvj:RobustScaler_gauss_adjust

Conversation

@mabelvj
Copy link
Copy Markdown
Contributor

@mabelvj mabelvj commented Nov 14, 2017

I made some changes in the function to implement gauss_adjust following the request in the thread Feature Request: Gaussian adjust option in RobustScaler #10139.

There is a new optional input parameter for RobustScaler called gauss_adjust. If gauss_adjust is set to True, the scale will be adjusted so the distribution of the output feature has std = 1. By default, gauss_adjust is set to False.

Also built the package and run the tests.

I'm new in open source projects, so let me know if there are any more steps I should follow.

@amueller
Copy link
Copy Markdown
Member

Sorry for no reply :( could you add a test that shows that this option makes StandardScaler and RobustScaler equivalent on normal data?

@cmarmo
Copy link
Copy Markdown
Contributor

cmarmo commented Jun 2, 2020

Closed as finalized in #17193.

@cmarmo cmarmo closed this Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Easy Well-defined and straightforward way to resolve module:preprocessing Waiting for Reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants