{"action":"created","issue":{"url":"https://api.github.com/repos/plotly/python-api/issues/259","labels_url":"https://api.github.com/repos/plotly/python-api/issues/259/labels{/name}","comments_url":"https://api.github.com/repos/plotly/python-api/issues/259/comments","events_url":"https://api.github.com/repos/plotly/python-api/issues/259/events","html_url":"https://github.com/plotly/python-api/pull/259","id":95061588,"number":259,"title":"add ohlc chart to TraceFactory","user":{"login":"cldougl","id":2789078,"avatar_url":"https://avatars.githubusercontent.com/u/2789078?v=3","gravatar_id":"","url":"https://api.github.com/users/cldougl","html_url":"https://github.com/cldougl","followers_url":"https://api.github.com/users/cldougl/followers","following_url":"https://api.github.com/users/cldougl/following{/other_user}","gists_url":"https://api.github.com/users/cldougl/gists{/gist_id}","starred_url":"https://api.github.com/users/cldougl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cldougl/subscriptions","organizations_url":"https://api.github.com/users/cldougl/orgs","repos_url":"https://api.github.com/users/cldougl/repos","events_url":"https://api.github.com/users/cldougl/events{/privacy}","received_events_url":"https://api.github.com/users/cldougl/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-07-14T22:59:30Z","updated_at":"2015-07-28T08:54:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/plotly/python-api/pulls/259","html_url":"https://github.com/plotly/python-api/pull/259","diff_url":"https://github.com/plotly/python-api/pull/259.diff","patch_url":"https://github.com/plotly/python-api/pull/259.patch"},"body":"- add ohlc chart as 2 TraceFactory methods so\n`TraceFactory.create_ohlc_increasing()` returns a trace of the\nincreasing data and `TraceFactory.create_ohlc_decreasing()` returns a\ntrace of the decreasing data. In these charts increasing and decreasing\nperiods are typically represented by different colours (default green &\nred).\n- pull flatten method out of `_Quiver` so other charts can call it"},"comment":{"url":"https://api.github.com/repos/plotly/python-api/issues/comments/125508917","html_url":"https://github.com/plotly/python-api/pull/259#issuecomment-125508917","issue_url":"https://api.github.com/repos/plotly/python-api/issues/259","id":125508917,"user":{"login":"theengineear","id":6611546,"avatar_url":"https://avatars.githubusercontent.com/u/6611546?v=3","gravatar_id":"","url":"https://api.github.com/users/theengineear","html_url":"https://github.com/theengineear","followers_url":"https://api.github.com/users/theengineear/followers","following_url":"https://api.github.com/users/theengineear/following{/other_user}","gists_url":"https://api.github.com/users/theengineear/gists{/gist_id}","starred_url":"https://api.github.com/users/theengineear/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theengineear/subscriptions","organizations_url":"https://api.github.com/users/theengineear/orgs","repos_url":"https://api.github.com/users/theengineear/repos","events_url":"https://api.github.com/users/theengineear/events{/privacy}","received_events_url":"https://api.github.com/users/theengineear/received_events","type":"User","site_admin":false},"created_at":"2015-07-28T08:54:51Z","updated_at":"2015-07-28T08:54:51Z","body":"@cldougl my main issue is that we now have `TraceFactory` and `FigureFactory`. I think this is more work for our users.\r\n\r\n@chriddyp this has been discussed before, but do we have a way of marking new functionality like this as *beta*? I feel like we could potentially get flak for changing this, but it's really not been out there for very long. When we do this in the api, we *at least* have [documentation](https://api.plot.ly/v2/#beta) about it. Perhaps just keeping something like BETA in the docstring would be a good idea for new things like this that are in development? IPython still says that the widgets framework is subject to change... and that's heavily used.\r\n\r\nI don't *think* we need to throw up a warning... but it would certainly get the point across..."}}