{"action":"created","issue":{"url":"https://api.github.com/repos/ioam/holoviews/issues/315","labels_url":"https://api.github.com/repos/ioam/holoviews/issues/315/labels{/name}","comments_url":"https://api.github.com/repos/ioam/holoviews/issues/315/comments","events_url":"https://api.github.com/repos/ioam/holoviews/issues/315/events","html_url":"https://github.com/ioam/holoviews/pull/315","id":119848274,"number":315,"title":"Added load_notebook_fn to replace %load_ext","user":{"login":"philippjfr","id":1550771,"avatar_url":"https://avatars.githubusercontent.com/u/1550771?v=3","gravatar_id":"","url":"https://api.github.com/users/philippjfr","html_url":"https://github.com/philippjfr","followers_url":"https://api.github.com/users/philippjfr/followers","following_url":"https://api.github.com/users/philippjfr/following{/other_user}","gists_url":"https://api.github.com/users/philippjfr/gists{/gist_id}","starred_url":"https://api.github.com/users/philippjfr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/philippjfr/subscriptions","organizations_url":"https://api.github.com/users/philippjfr/orgs","repos_url":"https://api.github.com/users/philippjfr/repos","events_url":"https://api.github.com/users/philippjfr/events{/privacy}","received_events_url":"https://api.github.com/users/philippjfr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-12-02T01:40:46Z","updated_at":"2015-12-02T10:37:27Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ioam/holoviews/pulls/315","html_url":"https://github.com/ioam/holoviews/pull/315","diff_url":"https://github.com/ioam/holoviews/pull/315.diff","patch_url":"https://github.com/ioam/holoviews/pull/315.patch"},"body":"Added a ParameterizedFunction to load the notebook resources and initialize the magics. Unlike %load_ext this version accepts arguments to include bokeh, suppress the logo and change the CSS settings such as the notebook width."},"comment":{"url":"https://api.github.com/repos/ioam/holoviews/issues/comments/161253586","html_url":"https://github.com/ioam/holoviews/pull/315#issuecomment-161253586","issue_url":"https://api.github.com/repos/ioam/holoviews/issues/315","id":161253586,"user":{"login":"jlstevens","id":890576,"avatar_url":"https://avatars.githubusercontent.com/u/890576?v=3","gravatar_id":"","url":"https://api.github.com/users/jlstevens","html_url":"https://github.com/jlstevens","followers_url":"https://api.github.com/users/jlstevens/followers","following_url":"https://api.github.com/users/jlstevens/following{/other_user}","gists_url":"https://api.github.com/users/jlstevens/gists{/gist_id}","starred_url":"https://api.github.com/users/jlstevens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jlstevens/subscriptions","organizations_url":"https://api.github.com/users/jlstevens/orgs","repos_url":"https://api.github.com/users/jlstevens/repos","events_url":"https://api.github.com/users/jlstevens/events{/privacy}","received_events_url":"https://api.github.com/users/jlstevens/received_events","type":"User","site_admin":false},"created_at":"2015-12-02T10:37:27Z","updated_at":"2015-12-02T10:37:27Z","body":"I'm also very happy with this. \r\n\r\nMy only concerns are is that we have no documentation/examples for the users. Fortunately, as the tests are passing and in this PR ``%load_ext`` is using ``load_notebook``, then the new behavior must match the old behavior meaning we could replace ``%load_ext`` with ``load_notebook()`` everywhere if we wanted to.\r\n\r\nThat said do we want to keep the old behavior (i.e matching what ``%load_ext`` used to do) shown in our notebooks?\r\n\r\nIf so, I think ``%%load_ext`` is still useful and valid and we can just mention the new ``hv.load_notebook`` in the text somewhere (probably across multiple tutorials).\r\n\r\nIf not, we should replace ``%load_notebook`` with ``load_notebook()`` where we feel the default behavior is improved and update all the notebooks to use it (which unfortunately might mean having to update all the tests).\r\n\r\nJust something to think about..."}}