Title: Hover labels and event data for cumulative histograms · Issue #2115 · plotly/plotly.js · GitHub
Open Graph Title: Hover labels and event data for cumulative histograms · Issue #2115 · plotly/plotly.js
X Title: Hover labels and event data for cumulative histograms · Issue #2115 · plotly/plotly.js
Description: In #2113 we're adding bin range information to histogram hover labels, and input point numbers to the event data. But CDFs were explicitly omitted from these changes because it's not so obvious how to handle them. My comment from that PR...
Open Graph Description: In #2113 we're adding bin range information to histogram hover labels, and input point numbers to the event data. But CDFs were explicitly omitted from these changes because it's not so obvious how...
X Description: In #2113 we're adding bin range information to histogram hover labels, and input point numbers to the event data. But CDFs were explicitly omitted from these changes because it's not so obv...
Opengraph URL: https://github.com/plotly/plotly.js/issues/2115
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Hover labels and event data for cumulative histograms","articleBody":"In #2113 we're adding bin range information to histogram hover labels, and input point numbers to the event data. But CDFs were explicitly omitted from these changes because it's not so obvious how to handle them.\r\n\r\nMy comment from that PR:\r\n\r\n\u003e this gets at some of the same issues that led to `cumulative.currentbin` but in principle a CDF shows the sum of all the data prior to a specific point, not the data within that bin. perhaps I should shift `p0` and `p1` depending on `currentbin` so that the hover label at least gets the description \"all the data prior to X\" precisely correct, even though it'll be a different value than the center of the bar...\r\n\u003e\r\n\u003e And then what about `pts`? You could say it should be included and again should be \"all the data prior to X\" but then it would be meaningless to select a single bar and not all the bars before it. Alternatively `pts` could contain \"all the data that was added in this bar\" which might be more useful in terms of selecting a single bar, but it's not really what that bar means.\r\n\r\nResponse from @etpinard:\r\n\r\n\u003e \u003e so that the hover label at least gets the description \"all the data prior to X\"\r\n\u003e \r\n\u003e Yeah. I think something along those lines would be useful down the road. \r\n\u003e \r\n\u003e \u003e You could say it should be included and again should be \"all the data prior to X\" but then it would be meaningless to select a single bar and not all the bars before it. Alternatively pts could contain \"all the data that was added in this bar\" \r\n\u003e \r\n\u003e Sounds to me like both _all-data-prior-to-X_ and _data-added-in-this-bar_ point lists could be useful, so we might have to emit both in the future.","author":{"url":"https://github.com/alexcjohnson","@type":"Person","name":"alexcjohnson"},"datePublished":"2017-10-24T15:36:18.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":3},"url":"https://github.com/2115/plotly.js/issues/2115"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:399f291c-59f0-ce09-8647-63f48f7d40cf |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | ED9C:12115C:34A2C9A:4AF4465:6A56A8C4 |
| html-safe-nonce | 6a0366ccf91fb5aeacb30291ac573c7418b2dcbf0cab984d632a5a6d7bde200c |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFRDlDOjEyMTE1QzozNEEyQzlBOjRBRjQ0NjU6NkE1NkE4QzQiLCJ2aXNpdG9yX2lkIjoiMTY5OTAxMzAwNDUwNTgxOTMzMiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | bf11c56f7fc0b590dff309fb11aef6dfffb1796bb2cc78be45cf747842e96cba |
| hovercard-subject-tag | issue:268081226 |
| github-keyboard-shortcuts | repository,issues,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/_view_fragments/issues/show/plotly/plotly.js/2115/issue_layout |
| twitter:image | https://opengraph.githubassets.com/ca39b7c1d37f54ce6f0bc3dffb2ece36267e061194d2d0a7727a52685a49e526/plotly/plotly.js/issues/2115 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/ca39b7c1d37f54ce6f0bc3dffb2ece36267e061194d2d0a7727a52685a49e526/plotly/plotly.js/issues/2115 |
| og:image:alt | In #2113 we're adding bin range information to histogram hover labels, and input point numbers to the event data. But CDFs were explicitly omitted from these changes because it's not so obvious how... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | alexcjohnson |
| hostname | github.com |
| expected-hostname | github.com |
| None | 13efe7fe8ecc41eda1078cbde8206041ed92bc54e29bcb737e9353dd9c407aad |
| turbo-cache-control | no-preview |
| 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 | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 5466ebe023a0abf876a1e6c5d7602ee8f5e6a02b |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width