| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Thanks, looks pretty good overall but I've got two general comments:
Sorry, something went wrong.
Co-authored-by: Bas van Beek <43369155+BvB93@users.noreply.github.com>
|
@BvB93 If I understood correctly, your comments should have been addressed now. I note that there are a bunch of functions in fromnumeric.py, where the where argument is not keyword-only, e.g. numpy/numpy/core/fromnumeric.py Line 2163 in 8f8e14e Wrt. the kw only inconsistency, would you like me to open an issue, a PR or both, or do nothing? |
Sorry, something went wrong.
So the move towards the use of keyword-only arguments is somewhat recent, and with this in mind it is no coincidence that the (keyword-only) where arguments addressed in this PR were only added back in numpy 1.20. Changing all the (much older) remaining where arguments to keyword-only would be technically be backwards compatible. While I doubt they're used much as a positional parameter out in the wild, it's probably not the risk of breaking things. |
Sorry, something went wrong.
|
Thanks @janusheid |
Sorry, something went wrong.
I read that as do nothing. I agree that such a change would be backwards incompatible and that one can argue that it is not terribly important. Sometimes it can be helpful to have an issue that others can find if they have the same question, but in this case it it perhaps more of a general problem of kw only args throughout the api. |
Sorry, something went wrong.
|
Hi-five on merging your first pull request to NumPy, @janusheide! We hope you stick around! Your choices aren’t limited to programming – you can review pull requests, help us stay on top of new and old issues, develop educational material, work on our website, add or improve graphic design, create marketing materials, translate website content, write grant proposals, and help with other fundraising initiatives. For more info, check out: https://numpy.org/contribute |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Adding typehints for 'where' argument in a couple of functions in fromnumeric, where it appears to be missing.