From 9e2ff2d3863a1e6ed4278972a1ce9a864bef8320 Mon Sep 17 00:00:00 2001
From: Christian Fufezan <christian@fufezan.net>
Date: Thu, 9 Jun 2016 15:04:10 +0200
Subject: [PATCH 1/3] Reduced plot size by outsourcing plotly.min.js

---
 plotly/offline/offline.py | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/plotly/offline/offline.py b/plotly/offline/offline.py
index 7a6df8eed40..ddb2a318a09 100644
--- a/plotly/offline/offline.py
+++ b/plotly/offline/offline.py
@@ -7,6 +7,7 @@
 
 import json
 import os
+import shutil
 import uuid
 import warnings
 from pkg_resources import resource_string
@@ -42,8 +43,9 @@ def download_plotlyjs(download_url):
 
 
 def get_plotlyjs():
-    path = os.path.join('offline', 'plotly.min.js')
-    plotlyjs = resource_string('plotly', path).decode('utf-8')
+    # path = os.path.join('offline', 'plotly.min.js')
+    # plotlyjs = resource_string('plotly', path).decode('utf-8')
+    plotlyjs = '</script><script src="plotly.min.js">'
     return plotlyjs
 
 
@@ -315,6 +317,14 @@ def plot(figure_or_data,
         ).format(id=plotdivid)
 
     if output_type == 'file':
+        src_path = os.path.join(
+            os.path.dirname(__file__), 'plotly.min.js'
+        )
+        # src_path = resource_string('plotly', path)
+        dest_path = os.path.join( os.path.dirname( filename ), 'plotly.min.js')
+        if os.path.exists( dest_path ) is False:
+            shutil.copy( src_path, dest_path )
+
         with open(filename, 'w') as f:
             if include_plotlyjs:
                 plotly_js_script = ''.join([

From 530b2993887510545082c57aeca40c68f60f1157 Mon Sep 17 00:00:00 2001
From: Christian Fufezan <christian@fufezan.net>
Date: Thu, 9 Jun 2016 15:13:35 +0200
Subject: [PATCH 2/3] Added offline kwarg outsource_plotly to allow for both
 kind of plotting, i.e. regular plotly with 1.2MB javascript in each plot, or
 having plotly.min.js in a seperate file

---
 plotly/offline/offline.py | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/plotly/offline/offline.py b/plotly/offline/offline.py
index ddb2a318a09..fb9af3073e4 100644
--- a/plotly/offline/offline.py
+++ b/plotly/offline/offline.py
@@ -42,10 +42,12 @@ def download_plotlyjs(download_url):
     pass
 
 
-def get_plotlyjs():
-    # path = os.path.join('offline', 'plotly.min.js')
-    # plotlyjs = resource_string('plotly', path).decode('utf-8')
-    plotlyjs = '</script><script src="plotly.min.js">'
+def get_plotlyjs( outsource_plotly = False ):
+    if outsource_plotly:
+        plotlyjs = '</script><script src="plotly.min.js">'
+    else:
+        path = os.path.join('offline', 'plotly.min.js')
+        plotlyjs = resource_string('plotly', path).decode('utf-8')
     return plotlyjs
 
 
@@ -244,7 +246,8 @@ def plot(figure_or_data,
          validate=True, output_type='file',
          include_plotlyjs=True,
          filename='temp-plot.html',
-         auto_open=True):
+         auto_open=True,
+         outsource_plotly=False):
     """ Create a plotly graph locally as an HTML document or string.
 
     Example:
@@ -290,6 +293,10 @@ def plot(figure_or_data,
     auto_open (default=True) -- If True, open the saved file in a
         web browser after saving.
         This argument only applies if `output_type` is 'file'.
+    outsource_plotly (default=False) -- If True, the javascript library will
+        be copied into a separate file, thereby reducing the file size for the
+        actual plat significantly. This is handy if several plots are created
+        in the same folder.
     """
     if output_type not in ['div', 'file']:
         raise ValueError(
@@ -329,7 +336,7 @@ def plot(figure_or_data,
             if include_plotlyjs:
                 plotly_js_script = ''.join([
                     '<script type="text/javascript">',
-                    get_plotlyjs(),
+                    get_plotlyjs(outsource_plotly=outsource_plotly),
                     '</script>',
                 ])
             else:
@@ -356,7 +363,7 @@ def plot(figure_or_data,
             return ''.join([
                 '<div>',
                 '<script type="text/javascript">',
-                get_plotlyjs(),
+                get_plotlyjs(outsource_plotly=outsource_plotly),
                 '</script>',
                 plot_html,
                 '</div>'

From ee3b430fbfacb77c938fc83bb82757a6734dd9ce Mon Sep 17 00:00:00 2001
From: Christian Fufezan <christian.x.fufezan@gsk.com>
Date: Mon, 6 Feb 2017 15:51:50 +0100
Subject: [PATCH 3/3] Added serve_plotlyjs_from_directory() instead of kwarg to
 separate html and js into two files

---
 plotly/offline/__init__.py |  3 ++-
 plotly/offline/offline.py  | 39 ++++++++++++++++++++++++++------------
 2 files changed, 29 insertions(+), 13 deletions(-)

diff --git a/plotly/offline/__init__.py b/plotly/offline/__init__.py
index f4a5fc2d2b6..085e8c7e7c4 100644
--- a/plotly/offline/__init__.py
+++ b/plotly/offline/__init__.py
@@ -10,5 +10,6 @@
     iplot,
     iplot_mpl,
     plot,
-    plot_mpl
+    plot_mpl,
+    serve_plotlyjs_from_directory
 )
diff --git a/plotly/offline/offline.py b/plotly/offline/offline.py
index fb9af3073e4..11845d2acc6 100644
--- a/plotly/offline/offline.py
+++ b/plotly/offline/offline.py
@@ -31,6 +31,7 @@
     _matplotlib_imported = False
 
 __PLOTLY_OFFLINE_INITIALIZED = False
+__SERVE_PLOTLYJS_FROM_DIRECTORY = False
 
 
 def download_plotlyjs(download_url):
@@ -42,8 +43,9 @@ def download_plotlyjs(download_url):
     pass
 
 
-def get_plotlyjs( outsource_plotly = False ):
-    if outsource_plotly:
+def get_plotlyjs():
+    global __SERVE_PLOTLYJS_FROM_DIRECTORY
+    if __SERVE_PLOTLYJS_FROM_DIRECTORY:
         plotlyjs = '</script><script src="plotly.min.js">'
     else:
         path = os.path.join('offline', 'plotly.min.js')
@@ -51,6 +53,19 @@ def get_plotlyjs( outsource_plotly = False ):
     return plotlyjs
 
 
+def serve_plotlyjs_from_directory():
+    """
+    Sets global plotly.offline.__SERVE_PLOTLYJS_FROM_DIRECTORY to True,
+    which will trigger the separation of the html and the plotly.min.js
+    into two files. This will reduce the overall space required
+    if muliple plots are created in the same directory, since only one
+    copy of plotly.min.js will be stored.
+    """
+    global __SERVE_PLOTLYJS_FROM_DIRECTORY
+    __SERVE_PLOTLYJS_FROM_DIRECTORY = True
+    return
+
+
 def init_notebook_mode(connected=False):
     """
     Initialize plotly.js in the browser if it hasn't been loaded into the DOM
@@ -247,7 +262,7 @@ def plot(figure_or_data,
          include_plotlyjs=True,
          filename='temp-plot.html',
          auto_open=True,
-         outsource_plotly=False):
+         ):
     """ Create a plotly graph locally as an HTML document or string.
 
     Example:
@@ -293,10 +308,6 @@ def plot(figure_or_data,
     auto_open (default=True) -- If True, open the saved file in a
         web browser after saving.
         This argument only applies if `output_type` is 'file'.
-    outsource_plotly (default=False) -- If True, the javascript library will
-        be copied into a separate file, thereby reducing the file size for the
-        actual plat significantly. This is handy if several plots are created
-        in the same folder.
     """
     if output_type not in ['div', 'file']:
         raise ValueError(
@@ -328,15 +339,19 @@ def plot(figure_or_data,
             os.path.dirname(__file__), 'plotly.min.js'
         )
         # src_path = resource_string('plotly', path)
-        dest_path = os.path.join( os.path.dirname( filename ), 'plotly.min.js')
-        if os.path.exists( dest_path ) is False:
-            shutil.copy( src_path, dest_path )
+        global __SERVE_PLOTLYJS_FROM_DIRECTORY
+        if __SERVE_PLOTLYJS_FROM_DIRECTORY:
+            dest_path = os.path.join(
+                os.path.dirname( filename ), 'plotly.min.js'
+            )
+            if os.path.exists( dest_path ) is False:
+                shutil.copy( src_path, dest_path )
 
         with open(filename, 'w') as f:
             if include_plotlyjs:
                 plotly_js_script = ''.join([
                     '<script type="text/javascript">',
-                    get_plotlyjs(outsource_plotly=outsource_plotly),
+                    get_plotlyjs(),
                     '</script>',
                 ])
             else:
@@ -363,7 +378,7 @@ def plot(figure_or_data,
             return ''.join([
                 '<div>',
                 '<script type="text/javascript">',
-                get_plotlyjs(outsource_plotly=outsource_plotly),
+                get_plotlyjs(),
                 '</script>',
                 plot_html,
                 '</div>'