Title: Make `functions` param to secondary_x/yaxis not keyword-only. by anntzer · Pull Request #28133 · matplotlib/matplotlib · GitHub
Open Graph Title: Make `functions` param to secondary_x/yaxis not keyword-only. by anntzer · Pull Request #28133 · matplotlib/matplotlib
X Title: Make `functions` param to secondary_x/yaxis not keyword-only. by anntzer · Pull Request #28133 · matplotlib/matplotlib
Description: I suspect the parameter was made keyword-only because the originally planned signature (#11859) was secondary_xaxis(location, forward=..., inverse=...) where the keywords actually add some semantics (though that signature overall seems worse); however, for a single functions parameter, having to type an extra functions= in the call doesn't help the reader much (either they know what secondary_x/yaxis does, in which case the explicit kwarg doesn't matter, or they don't, in which case the kwarg name hardly helps)... and is a bit annoying. See the modified gallery entry, for example. PR summary PR checklist "closes #0000" is in the body of the PR description to link the related issue new and changed code is tested Plotting related features are demonstrated in an example New Features and API Changes are noted with a directive and release note Documentation complies with general and docstring guidelines
Open Graph Description: I suspect the parameter was made keyword-only because the originally planned signature (#11859) was secondary_xaxis(location, forward=..., inverse=...) where the keywords actually add some semantic...
X Description: I suspect the parameter was made keyword-only because the originally planned signature (#11859) was secondary_xaxis(location, forward=..., inverse=...) where the keywords actually add some semantic...
Opengraph URL: https://github.com/matplotlib/matplotlib/pull/28133
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:c4a23a49-7167-d9f3-7fce-0a906a479611 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | B86E:234B42:1E195D0:2913DD5:6A540AF1 |
| html-safe-nonce | b8dc0f92e1e4c657107e435ab32207def713e164c10fbde892359208dcef782f |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCODZFOjIzNEI0MjoxRTE5NUQwOjI5MTNERDU6NkE1NDBBRjEiLCJ2aXNpdG9yX2lkIjoiMjExMzAxNDEzNjM0Njc3NDI1NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 628be148a7f07fa488189b1cafbaf02fafc3e19fd887136e79453e4bef76798d |
| hovercard-subject-tag | pull_request:1839578942 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/matplotlib/matplotlib/pull/28133/files |
| twitter:image | https://avatars.githubusercontent.com/u/1322974?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/1322974?s=400&v=4 |
| og:image:alt | I suspect the parameter was made keyword-only because the originally planned signature (#11859) was secondary_xaxis(location, forward=..., inverse=...) where the keywords actually add some semantic... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/matplotlib/matplotlib git https://github.com/matplotlib/matplotlib.git |
| octolytics-dimension-user_id | 215947 |
| octolytics-dimension-user_login | matplotlib |
| octolytics-dimension-repository_id | 1385122 |
| octolytics-dimension-repository_nwo | matplotlib/matplotlib |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1385122 |
| octolytics-dimension-repository_network_root_nwo | matplotlib/matplotlib |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 07a982c1d40157c619b364352b704c3ce66bb332 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width