From 5a3fc0a4d66b70429d10abd43fe8581dc8163cfc Mon Sep 17 00:00:00 2001 From: martinRenou Date: Thu, 22 Aug 2024 11:09:29 +0200 Subject: [PATCH] Remove confusing error message This error message was introduced in https://github.com/jupyter-widgets/ipywidgets/pull/3335. It shows up when the user refreshes the JupyterLab page on a kernel that has not yet imported ipywidgets, because the manager requests for widgets states, but the kernel is not able to answer back any widget model as ipwidgets was not imported. It is a normal workflow that should not show any error message. This error message would also show up when using the manager with a kernel using ipywidgets 7, in that case as well we should not show any error message as it's a normal workflow too. This PR removes the error message to prevent any confusion from the user. --- packages/base-manager/src/manager-base.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/packages/base-manager/src/manager-base.ts b/packages/base-manager/src/manager-base.ts index 4953767894..c8e3ee9964 100644 --- a/packages/base-manager/src/manager-base.ts +++ b/packages/base-manager/src/manager-base.ts @@ -427,10 +427,6 @@ export abstract class ManagerBase implements IWidgetManager { initComm.close(); } catch (error) { - console.warn( - 'Failed to fetch ipywidgets through the "jupyter.widget.control" comm channel, fallback to fetching individual model state. Reason:', - error - ); // Fall back to the old implementation for old ipywidgets backend versions (ipywidgets<=7.6) return this._loadFromKernelModels(); }