Skip to content

DEPR Rename keyword "quantile" to "q" in Rolling.quantile #53216

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 17, 2023

Conversation

Charlie-XIAO
Copy link
Contributor

@Charlie-XIAO Charlie-XIAO commented May 13, 2023

@phofl phofl added Deprecate Functionality to remove in pandas Window rolling, ewma, expanding labels May 14, 2023
@phofl phofl added this to the 2.1 milestone May 14, 2023
@mroeschke
Copy link
Member

Could you deprecate this on Expanding too?

@mroeschke
Copy link
Member

In rolling.py there's also this "private" quantile function

    def quantile(
        self,
        quantile: float,
        interpolation: QuantileInterpolation = "linear",
        numeric_only: bool = False,
    ):

could you change quantile to q here too?

@Charlie-XIAO
Copy link
Contributor Author

In rolling.py there's also this "private" quantile function, could you change quantile to q here too?

Sure, and since it is a private function, no deprecation is needed right?

@mroeschke
Copy link
Member

Sure, and since it is a private function, no deprecation is needed right?

Correct

@mroeschke mroeschke merged commit 199624d into pandas-dev:main May 17, 2023
@mroeschke
Copy link
Member

Thanks @Charlie-XIAO

topper-123 pushed a commit to topper-123/pandas that referenced this pull request May 22, 2023
…ev#53216)

* deprecated 'quantile' arg in Rolling.quantile

* renamed keyword 'quantile' to 'q' in Rolling.quantile

* fixed test

* docstring modified, removed unnecessary descriptions and typos

* resolved conversations

* deprecate on Expanding as well

* resolved conversations; renamed in parent
@Charlie-XIAO Charlie-XIAO deleted the depr-rolling-q branch June 28, 2023 05:42
Daquisu pushed a commit to Daquisu/pandas that referenced this pull request Jul 8, 2023
…ev#53216)

* deprecated 'quantile' arg in Rolling.quantile

* renamed keyword 'quantile' to 'q' in Rolling.quantile

* fixed test

* docstring modified, removed unnecessary descriptions and typos

* resolved conversations

* deprecate on Expanding as well

* resolved conversations; renamed in parent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Deprecate Functionality to remove in pandas Window rolling, ewma, expanding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ENH: Differing variable name for quantile vs rolling.quantile
3 participants