{"action":"created","issue":{"url":"https://api.github.com/repos/bokeh/bokeh/issues/5037","repository_url":"https://api.github.com/repos/bokeh/bokeh","labels_url":"https://api.github.com/repos/bokeh/bokeh/issues/5037/labels{/name}","comments_url":"https://api.github.com/repos/bokeh/bokeh/issues/5037/comments","events_url":"https://api.github.com/repos/bokeh/bokeh/issues/5037/events","html_url":"https://github.com/bokeh/bokeh/pull/5037","id":173257369,"number":5037,"title":"Exposing tooltips columns in Charts API","user":{"login":"grromrell","id":6600690,"avatar_url":"https://avatars.githubusercontent.com/u/6600690?v=3","gravatar_id":"","url":"https://api.github.com/users/grromrell","html_url":"https://github.com/grromrell","followers_url":"https://api.github.com/users/grromrell/followers","following_url":"https://api.github.com/users/grromrell/following{/other_user}","gists_url":"https://api.github.com/users/grromrell/gists{/gist_id}","starred_url":"https://api.github.com/users/grromrell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grromrell/subscriptions","organizations_url":"https://api.github.com/users/grromrell/orgs","repos_url":"https://api.github.com/users/grromrell/repos","events_url":"https://api.github.com/users/grromrell/events{/privacy}","received_events_url":"https://api.github.com/users/grromrell/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/bokeh/bokeh/labels/status:%20WIP","name":"status: WIP","color":"fef2c0"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":12,"created_at":"2016-08-25T17:00:08Z","updated_at":"2016-09-10T16:00:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bokeh/bokeh/pulls/5037","html_url":"https://github.com/bokeh/bokeh/pull/5037","diff_url":"https://github.com/bokeh/bokeh/pull/5037.diff","patch_url":"https://github.com/bokeh/bokeh/pull/5037.patch"},"body":"issues: fixes #4347 \r\n\r\nCharts have a kw argument for tooltips, but if the columns do not exist then the hover tool will display '???' This change will find columns that are specified by the tooltips kw argument and add them to the glyph data source if they do not already exist. Currently only the ScatterBuilder is affected. If the change is amenable as supplied I can work on adding the solution to the other Builders."},"comment":{"url":"https://api.github.com/repos/bokeh/bokeh/issues/comments/246119646","html_url":"https://github.com/bokeh/bokeh/pull/5037#issuecomment-246119646","issue_url":"https://api.github.com/repos/bokeh/bokeh/issues/5037","id":246119646,"user":{"login":"bryevdv","id":1078448,"avatar_url":"https://avatars.githubusercontent.com/u/1078448?v=3","gravatar_id":"","url":"https://api.github.com/users/bryevdv","html_url":"https://github.com/bryevdv","followers_url":"https://api.github.com/users/bryevdv/followers","following_url":"https://api.github.com/users/bryevdv/following{/other_user}","gists_url":"https://api.github.com/users/bryevdv/gists{/gist_id}","starred_url":"https://api.github.com/users/bryevdv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bryevdv/subscriptions","organizations_url":"https://api.github.com/users/bryevdv/orgs","repos_url":"https://api.github.com/users/bryevdv/repos","events_url":"https://api.github.com/users/bryevdv/events{/privacy}","received_events_url":"https://api.github.com/users/bryevdv/received_events","type":"User","site_admin":false},"created_at":"2016-09-10T16:00:00Z","updated_at":"2016-09-10T16:00:00Z","body":"Also, as an idea, I wonder if it might not be better (possibly longer term) to have the `ToolTip` have a method that can return a list of all the columns that it will refer to. That could be used for validation at lower levels, and for `bokeh.charts`, simple charts like `Scatter` could take that list and then use it to copy columns into the JSON for the chart (as yo are doing now). But more complicated charts like `Bar` could take that list and potentially do similar aggregations that were done for the bars "}}
{"actor":{"display_login":"bryevdv"}}