[issue38308] Add optional weighting to statistics.harmonic_mean()

Serhiy Storchaka added the comment: New changeset 66136768615472a8d1a18b5018095b9737dbab8c by Zackery Spytz in branch 'master': bpo-38308: Fix the "versionchanged" for the *weights* of harmonic_mean() (GH-23919)

[issue38308] Add optional weighting to statistics.harmonic_mean()

Zackery Spytz added the comment: The "versionchanged" for *weights* should be 3.10, not 3.8. I've created PR 23919 to fix this. -- versions: +Python 3.10 -Python 3.9 ___ Python tracker

[issue38308] Add optional weighting to statistics.harmonic_mean()

Change by Zackery Spytz : -- nosy: +ZackerySpytz nosy_count: 4.0 -> 5.0 pull_requests: +22770 pull_request: https://github.com/python/cpython/pull/23919 ___ Python tracker ___

[issue38308] Add optional weighting to statistics.harmonic_mean()

Raymond Hettinger added the comment: Thank you and Merry Christmas. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue38308] Add optional weighting to statistics.harmonic_mean()

Raymond Hettinger added the comment: New changeset cc3467a57b61b0e7ef254b36790a1c44b13f2228 by Raymond Hettinger in branch 'master': bpo-38308: Add optional weighting to statistics.harmonic_mean() (GH-23914) https://github.com/python/cpython/commit/cc3467a57b61b0e7ef254b36790a1c44b13f2228

[issue38308] Add optional weighting to statistics.harmonic_mean()

Steven D'Aprano added the comment: Okay, I'm satisfied with that reasoning, thanks Raymond. Patch looks good to me. Go for it! Have a good Christmas and stay safe. -- ___ Python tracker

[issue38308] Add optional weighting to statistics.harmonic_mean()

Raymond Hettinger added the comment: > I would like to hear why you removed the P/E example > instead of just adding additional examples. I tried out the existing P/E example in my Python courses and found that it had very little explanatory power — in general, non-finance people know less

[issue38308] Add optional weighting to statistics.harmonic_mean()

Steven D'Aprano added the comment: I like the addition but I'm not sure why you removed the price-earnings ratio example from the docs. I think that it's useful to have an example that shows that harmonic mean is not *just* for speed-related problems. I'm not going to reject your change

[issue38308] Add optional weighting to statistics.harmonic_mean()

Change by Raymond Hettinger : -- assignee: steven.daprano -> rhettinger ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38308] Add optional weighting to statistics.harmonic_mean()

Change by Raymond Hettinger : -- keywords: +patch pull_requests: +22765 stage: -> patch review pull_request: https://github.com/python/cpython/pull/23914 ___ Python tracker

[issue38308] Add optional weighting to statistics.harmonic_mean()

Dong-hee Na added the comment: Oh.. Thank you for letting me know -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38308] Add optional weighting to statistics.harmonic_mean()

Raymond Hettinger added the comment: > Can I process with this issue with your guide? Thank you, but this is one I would like to do myself. I've already done work on it and would like to wrap it up (also, it's more complicated than it seems because the supporting functions are a bit

[issue38308] Add optional weighting to statistics.harmonic_mean()

Dong-hee Na added the comment: @rhettinger If you are okay, Can I process with this issue with your guide? -- ___ Python tracker ___

[issue38308] Add optional weighting to statistics.harmonic_mean()

Dong-hee Na added the comment: Great idea! -- nosy: +corona10 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue38308] Add optional weighting to statistics.harmonic_mean()

Steven D'Aprano added the comment: Sounds like a great idea to me. Thanks, Steven -- ___ Python tracker ___ ___ Python-bugs-list

[issue38308] Add optional weighting to statistics.harmonic_mean()

Change by Mark Dickinson : -- nosy: +mark.dickinson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue38308] Add optional weighting to statistics.harmonic_mean()

New submission from Raymond Hettinger : Currently, harmonic_mean() is difficult to use in real applications because it assumes equal weighting. While that is sometimes true, the API precludes a broad class of applications where the weights are uneven. That is easily remedied with an optional