[proxy] web.archive.org — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public
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

bpo-36546: Fix typo quaatile to quantile #13001

Merged
merged 1 commit into from May 2, 2019
Merged

bpo-36546: Fix typo quaatile to quantile #13001

merged 1 commit into from May 2, 2019

Conversation

Copy link
Member

tirkarthi commented Apr 29, 2019 •
edited by bedevere-bot

bedevere-bot added tests Tests in the Lib/test dir awaiting review labels Apr 29, 2019
rougeth reviewed Apr 29, 2019
View reviewed changes
Lib/test/test_statistics.py
@@ -2178,7 +2178,7 @@ def f(x):
def test_specific_cases_inclusive(self):
# Match results computed by hand and cross-checked
# against the PERCENTILE.INC function in MS Excel
# and against the quaatile() function in SciPy.
# and against the quantile() function in SciPy.
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't be better to reference NumPy instead of SciPy?
ref: https://docs.scipy.org/doc/numpy/reference/generated/numpy.quantile.html

Copy link
Member Author

Choose a reason for hiding this comment

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

I will leave it to @rhettinger on changing scipy to numpy

rhettinger self-assigned this May 2, 2019
rhettinger merged commit 874ad1b into python:master May 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

rougeth rougeth approved these changes

Assignees

rhettinger

Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants