Skip to content

Commit 69efca9

Browse files
authored
Remove confusing error message (#3936)
This error message was introduced in #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.
1 parent 0aa1efb commit 69efca9

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

packages/base-manager/src/manager-base.ts

-4
Original file line numberDiff line numberDiff line change
@@ -427,10 +427,6 @@ export abstract class ManagerBase implements IWidgetManager {
427427

428428
initComm.close();
429429
} catch (error) {
430-
console.warn(
431-
'Failed to fetch ipywidgets through the "jupyter.widget.control" comm channel, fallback to fetching individual model state. Reason:',
432-
error
433-
);
434430
// Fall back to the old implementation for old ipywidgets backend versions (ipywidgets<=7.6)
435431
return this._loadFromKernelModels();
436432
}

0 commit comments

Comments
 (0)