From 61ddcd242c69eee5a123f79bba1241dca3841654 Mon Sep 17 00:00:00 2001 From: chriddyp Date: Mon, 30 Nov 2015 22:13:05 -0500 Subject: [PATCH 1/3] :bug: get the correct plotly_domain, not the "session"'s --- plotly/offline/offline.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/plotly/offline/offline.py b/plotly/offline/offline.py index 24e08340bee..cb9e3762bed 100644 --- a/plotly/offline/offline.py +++ b/plotly/offline/offline.py @@ -11,7 +11,8 @@ import warnings from pkg_resources import resource_string -from plotly import session, tools, utils +import plotly +from plotly import tools, utils from plotly.exceptions import PlotlyError @@ -129,8 +130,10 @@ def iplot(figure_or_data, show_link=True, link_text='Export to plot.ly', config['linkText'] = link_text jconfig = json.dumps(config) - plotly_platform_url = session.get_session_config().get('plotly_domain', - 'https://plot.ly') + # TODO: The get_config 'source of truth' should + # really be somewhere other than plotly.plotly + plotly_platform_url = plotly.plotly.get_config().get('plotly_domain', + 'https://plot.ly') if (plotly_platform_url != 'https://plot.ly' and link_text == 'Export to plot.ly'): From 70c78853d09220a4590ce61d3a30a9a070a12095 Mon Sep 17 00:00:00 2001 From: chriddyp Date: Mon, 30 Nov 2015 22:23:49 -0500 Subject: [PATCH 2/3] :fireworks: V1.9.2 - export to plotly fix for plotly offline mode --- CHANGELOG.md | 11 +++++++++++ plotly/version.py | 2 +- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0f97382ec45..eca55e8d7bd 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,17 @@ This project adheres to [Semantic Versioning](http://semver.org/). ## [Unreleased] +## [1.9.2] - 2015-11-30 +**Bug Fix**: Previously, the "Export to plot.ly" link on +offline charts would export your figures to the +public plotly cloud, even if your `config_file` +(set with `plotly.tools.set_config_file` to the file +`~/.plotly/.config`) set `plotly_domain` to a plotly enterprise +URL like `https://plotly.acme.com`. + +This is now fixed. Your graphs will be exported to your +`plotly_domain` if it is set. + ## [1.9.1] - 2015-11-26 ### Added - The FigureFactory can now create annotated heatmaps with `.create_annotated_heatmap`. diff --git a/plotly/version.py b/plotly/version.py index 35424e875ea..4175d399802 100644 --- a/plotly/version.py +++ b/plotly/version.py @@ -1 +1 @@ -__version__ = '1.9.1' +__version__ = '1.9.2' From 5bebbd2d59a3aa713f0d81a35d58d9f26bef8c34 Mon Sep 17 00:00:00 2001 From: chriddyp Date: Mon, 30 Nov 2015 22:24:09 -0500 Subject: [PATCH 3/3] :dolls: Add some examples to the changelog --- CHANGELOG.md | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index eca55e8d7bd..839f3209139 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -17,8 +17,16 @@ This is now fixed. Your graphs will be exported to your ## [1.9.1] - 2015-11-26 ### Added -- The FigureFactory can now create annotated heatmaps with `.create_annotated_heatmap`. +- The FigureFactory can now create annotated heatmaps with `.create_annotated_heatmap`. Check it out with: +``` +import plotly.tools as tls +help(tls.FigureFactory.create_annotated_heatmap) +``` - The FigureFactory can now create tables with `.create_table`. +``` +import plotly.tools as tls +help(tls.FigureFactory.create_table) +``` ## [1.9.0] - 2015-11-15 - Previously, using plotly offline required a paid license.