Remove comm_info workaround

Author: SylvainCorlay

ipywidgets/__init__.py

@@ -35,15 +35,3 @@ def _handle_ipython():
     load_ipython_extension(ip)
 
 _handle_ipython()
-
-
-# Workaround for the absence of a comm_info_[request/reply] shell message
-class CommInfo(Widget):
-    """CommInfo widgets are is typically instantiated by the front-end. As soon as it is instantiated, it sends the collection of valid comms, and kills itself. It is a workaround to the absence of comm_info shell message."""
-
-    def __init__(self, **kwargs):
-        super(CommInfo, self).__init__(**kwargs)
-        self.send(dict(comms={
-            k: dict(target_name=v.target_name) for (k, v) in self.comm.kernel.comm_manager.comms.items()
-        }))
-        self.close()

ipywidgets/static/widgets/js/manager.js

@@ -510,30 +510,13 @@ define([
 
     WidgetManager.prototype._get_comm_info = function(kernel) {
         /**
-         * Gets a promise for the open comms in the backend
+         * Gets a promise for the valid widget models.
          */
-
-        // Version using the comm_list_[request/reply] shell message.
-        /*var that = this;
+        var that = this;
         return new Promise(function(resolve, reject) {
-             kernel.comm_info(function(msg) {
+             kernel.comm_info('ipython.widget', function(msg) {
                  resolve(msg['content']['comms']);
              });
-        });*/
-
-        // Workaround for absence of comm_list_[request/reply] shell message.
-        // Create a new widget that gives the comm list and commits suicide.
-        var that = this;
-        return new Promise(function(resolve, reject) {
-            var comm = that.comm_manager.new_comm('ipython.widget',
-                                {'widget_class': 'ipywidgets.CommInfo'},
-                                'comm_info');
-            comm.on_msg(function(msg) {
-                var data = msg.content.data;
-                if (data.content && data.method === 'custom') {
-                    resolve(data.content.comms);
-                }
-            });
         });
     };
 

setup.py

@@ -276,8 +276,8 @@ def run(self):
 
 setuptools_args = {}
 install_requires = setuptools_args['install_requires'] = [
-    'ipython>=4.0.0dev0',
-    'ipykernel',
+    'ipython>=4.0.0',
+    'ipykernel>=4.1.0',
     'traitlets',
     'notebook',
 ]