Title: Optimize 3D data limits calculation by simplifying direct updates to dataLim by Thierno88 · Pull Request #28998 · matplotlib/matplotlib · GitHub
Open Graph Title: Optimize 3D data limits calculation by simplifying direct updates to dataLim by Thierno88 · Pull Request #28998 · matplotlib/matplotlib
X Title: Optimize 3D data limits calculation by simplifying direct updates to dataLim by Thierno88 · Pull Request #28998 · matplotlib/matplotlib
Description: This pull request addresses the inefficiency in updating 3D dataLim for plots in Matplotlib by eliminating unnecessary data processing and copying. Instead of creating a new _Bbox3d class, this PR introduces methods to directly compute and update the X, Y, and Z data limits using individual functions for each axis (get_datalim_x, get_datalim_y, get_datalim_z). This simplifies the code and improves performance by reducing the overhead of manipulating large data structures. The change allows for direct updates of dataLim within the plotting logic, making the process of auto-scaling more efficient. This resolves the same core problem as discussed in issue #28403 but with a different approach, focusing on minimal code changes and improved performance. PR Checklist: closes #28403 New and changed code is tested to ensure accuracy and robustness.
Open Graph Description: This pull request addresses the inefficiency in updating 3D dataLim for plots in Matplotlib by eliminating unnecessary data processing and copying. Instead of creating a new _Bbox3d class, this PR ...
X Description: This pull request addresses the inefficiency in updating 3D dataLim for plots in Matplotlib by eliminating unnecessary data processing and copying. Instead of creating a new _Bbox3d class, this PR ...
Opengraph URL: https://github.com/matplotlib/matplotlib/pull/28998
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:2535b6ac-c6fd-550b-9233-da9b104ff418 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | BDE2:166AAB:1F6FAA7:2ACB0EB:6A53825A |
| html-safe-nonce | 4cb606751d49cbf7f91056440a6798d53a7d9a6c29ab1da2fc7c4c1f1c5c384e |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCREUyOjE2NkFBQjoxRjZGQUE3OjJBQ0IwRUI6NkE1MzgyNUEiLCJ2aXNpdG9yX2lkIjoiNjc0ODEzODEyOTQ4NTY2NjE4IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 5e47b216a9667f028aed7298612123a55cc50dde11b50bc2e81b0659cf54f285 |
| hovercard-subject-tag | pull_request:2134742190 |
| 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/28998/files |
| twitter:image | https://avatars.githubusercontent.com/u/82781367?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/82781367?s=400&v=4 |
| og:image:alt | This pull request addresses the inefficiency in updating 3D dataLim for plots in Matplotlib by eliminating unnecessary data processing and copying. Instead of creating a new _Bbox3d class, this PR ... |
| 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