allows plotly plots to be optionally placed in another div

[hottwaj]

allows plotly plots to be optionally placed in another div by specifying the div id as "div_id" in the MIME json package

If present, the target div id is passed to plotly which takes care of rendering the chart in the given div

implements #173

Perhaps this is not of interest to anyone else and I'm happy to just keep these changes in my own fork, so feel free to reject this PR.

In case its of interest the reason I quite like using this is to create more complex chart layouts by first setting out a set of divs or a table to contain the charts, and then rendering all the charts

e.g.

class PlotlyMIMEBundle(object):
    def __init__(self, data=[], layout={}, div_id=None):
        bundle = {}
        bundle['application/vnd.plotly.v1+json'] = { 
            'data': data, 
            'layout': layout, 
            'div_id': div_id
        }
        self.bundle = bundle
    
    def _repr_mimebundle_(self, *args, **kwargs):
        return self.bundle

display(HTML("<div id="chart1"></div><div id="chart2"></div>"))
display(PlotlyMIMEBundle(data = [{"x": [1, 2, 3], "y": [3, 1, 6]}], div_id = "chart1"))
display(PlotlyMIMEBundle(data = [{"x": [5, 4, 3], "y": [3, 1, 6]}], div_id = "chart2"))

Thanks!

[ian-r-rose]

Sorry for the very slow response @hottwaj! The plotly extension in this repo has now been deprecated in favor of the official jupyterlab-plotly extension developed by Plotly. Does your fix here also apply to that extension? If so, would you mind opening the same PR over there?

[hottwaj]

Thanks for the heads up Ian and sorry for the slow response on my side! I'll check if it's needed in the new repo and go from there. Cheers

…

On Wed, 18 Sep 2019, 04:57 Ian Rose, ***@***.***> wrote: Sorry for the very slow response @hottwaj <https://github.com/hottwaj>! The plotly extension in this repo has now been deprecated in favor of the official jupyterlab-plotly extension developed by Plotly. Does your fix here also apply to that extension? If so, would you mind opening the same PR over there? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#178?email_source=notifications&email_token=ABK6JBITLCJGKGVKQBORHTDQKGRMDA5CNFSM4GWX7JE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD66W3RA#issuecomment-532508100>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABK6JBOTT4WQEKKBZVZMBOLQKGRMDANCNFSM4GWX7JEQ> .

[fcollonval]

Closing as plotly is maintaining officially its renderer: https://github.com/plotly/plotly.py