Title: Optimize the text measuring of legend items when rendering large amount of traces by hy9be · Pull Request #1586 · plotly/plotly.js · GitHub
Open Graph Title: Optimize the text measuring of legend items when rendering large amount of traces by hy9be · Pull Request #1586 · plotly/plotly.js
X Title: Optimize the text measuring of legend items when rendering large amount of traces by hy9be · Pull Request #1586 · plotly/plotly.js
Description: I open this as a separate PR from #1585, because I am not sure if you guys will be willing to take this one. The logic behind my change here is that if you want to plot reasonable amount of traces, we will give one very precise measuring and layout on legend items. But if you go crazy, we will sacrifice the precision of legend item layout (which probably does not matter anymore in this case) for way much better performance. On my machine, this change reduced the rendering time for 100 traces (200 points per trace) from 1.6 seconds to 1.1 seconds 300 traces (200 points per trace) from 5.7 seconds to 2.8 seconds 500 traces (200 points per trace) from 12.4 seconds to 5.8 seconds
Open Graph Description: I open this as a separate PR from #1585, because I am not sure if you guys will be willing to take this one. The logic behind my change here is that if you want to plot reasonable amount of traces,...
X Description: I open this as a separate PR from #1585, because I am not sure if you guys will be willing to take this one. The logic behind my change here is that if you want to plot reasonable amount of traces,...
Opengraph URL: https://github.com/plotly/plotly.js/pull/1586
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:214c413d-e796-7d34-c621-7ba5ac7d62bc |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | CC1E:1B5602:472AE9F:5D4B6DB:6A5D6C60 |
| html-safe-nonce | adc8e607bae30af115d664bbea5198fa9087bf142ecbb9f5a906eb0cbe7d846a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDQzFFOjFCNTYwMjo0NzJBRTlGOjVENEI2REI6NkE1RDZDNjAiLCJ2aXNpdG9yX2lkIjoiODY2MDYyNDMyMjM0MzAzODA0OCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | dc2af00b58f4482b7b65eca7b290a355ae8215a730d269978fa40a06b4ca4105 |
| hovercard-subject-tag | pull_request:115437885 |
| 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/plotly/plotly.js/pull/1586/files |
| twitter:image | https://avatars.githubusercontent.com/u/3392907?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/3392907?s=400&v=4 |
| og:image:alt | I open this as a separate PR from #1585, because I am not sure if you guys will be willing to take this one. The logic behind my change here is that if you want to plot reasonable amount of traces,... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5290d7e14309ad1e76106a9c4237bd1041517e83ea182c8ab756752cb0c6940b |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/plotly/plotly.js git https://github.com/plotly/plotly.js.git |
| octolytics-dimension-user_id | 5997976 |
| octolytics-dimension-user_login | plotly |
| octolytics-dimension-repository_id | 45646037 |
| octolytics-dimension-repository_nwo | plotly/plotly.js |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 45646037 |
| octolytics-dimension-repository_network_root_nwo | plotly/plotly.js |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 9c975978430e9ad293956f2bbdaf153b1bd84a99 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width