-
Notifications
You must be signed in to change notification settings - Fork 299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unable to open Notebooks in Insiders #5804
Comments
Thanks for filling this issue, I'll look into this asap. |
jupyter.opennotebook
not found
Hmm, looks like theres a bug in VS Code microsoft/vscode#123307 |
Yup, i too can replicate this. |
This should be fixed in the latest VS Code Insiders build: Version: 1.57.0-insider (user setup) @sztal could you please update VS Code Insiders and let us know if the issue persists? Thanks and sorry for the trouble! |
Hi, many thanks for such a quick reaction! However, it seems the problem still persists. I updated VS Code correctly, I think (below is the version info I get after updating):
The errors I see in Developer Tools seems to be identical. And crucially, when I inspect settings I still see that:
is marked with a warning |
The same issue is affecting me too. |
@sztal To be clear, you cannot open notebooks even after upgrading? The warning is benign. TLDR VS Code has changed the layout for this setting, but internally they are supposed to still recognize the old format (though we are likely going to upgrade to the new layout, see #5806). There was a bug that caused their backward compatibility code to fail which we're told has since been fixed. But if you can't open notebooks at all then the problem is somewhere else. |
@Hanuphant, are you on the latest VS Code Insiders? There was an update that shipped 3 hours ago which should have resolved this. |
Yes, it is still not working. Updated 10 mins ago. |
@joyceerhl , yes, I still cannot open any notebook. In some cases they are opened as plain text .json files and in some cases VS Code just hangs while trying to open them. And also, all the errors I mentioned earlier are still there in the DevTools Console. In particular, there is this one which looks probably most worrying (raised at
|
Currently, from the dev view there are 4 errors and 3 warnings shown now. The update did fix a 5th one which appeared earlier in the day. workbench.desktop.main.js:1799 Unable to instantiate workbench contribution f. TypeError: Cannot read property 'windowsExec' of undefined at f._refreshOpenInTerminalMenuItemTitle (workbench.desktop.main.js:1973) at new f (workbench.desktop.main.js:1973) at g._createInstance (workbench.desktop.main.js:625) at g.createInstance (workbench.desktop.main.js:625) at S.safeCreateInstance (workbench.desktop.main.js:1799) at S.doInstantiateByPhase (workbench.desktop.main.js:1799) at workbench.desktop.main.js:1799
(node:17900) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead. Extension activation failed, run the 'Developer: Toggle Developer Tools' command for more information. workbench.desktop.main.js:62 [Extension Host] extension activation failed TypeError: o.getConfiguration(...).get(...).find is not a function The aforementioned are the errors I am seeing now. |
Yep I repro the bug. Assigning to myself for investigation. Thanks for letting us know. |
@DonJayamanne It's working for you because your settings.json still contains the old editorAssociations setting format. If you delete it, this code here falls over and crashes activation: vscode-jupyter/src/client/common/experiments/service.ts Lines 64 to 70 in 94ff177
|
As a temporary workaround @sztal and @Hanuphant, you can replace the workbench.editorAssociations setting with the following:
I assume both of you have intentionally opted out of the native notebooks UI. However this will allow the Jupyter extension to activate. We're working on a fix for this (basically we need to support reading the new setting format that VS Code has introduced, which is the default layout now). |
Thanks! But now I am a bit confused. I did not want to opt out from the native notebooks UI. As a matter of fact it was the reason I started to use VS Code Insiders in the first place. So does it mean that I could have opted out somehow accidentally? And if so, how can I check this? |
Do you have The "jupyter.notebook.ipynb" viewType is for our custom editor implementation of notebooks. So you shouldn't have that entry there. The "jupyter-notebook" viewType is for our native notebooks implementation. |
But in the meantime, I confirm that your solution works (and I still have new native notebooks). I think that this So once again, thanks a lot for solving it so quickly! |
Yeah, I did have I am quite new to this, will you be able to help me out with workbench.editorAssociation . Do I have to add that code in settings.json? |
Yup, you would delete the existing "workbench.editorAssociations" key and value, and replace it with this snippet:
|
Thanks a lot @joyceerhl & devs. It worked!!! |
I too was using Insider for the native view capability. |
@flipphillips are you also unable to open notebooks in insiders? Does this workaround help?: #5804 (comment) |
@joyceerhl - I see what the problem is/was, that the settings format is different now than what was in the synced settings (I had other associations in there too). It appears that this manual fix worked, thanks! |
Yeah the new VS Code core setting IntelliSense flags the old format as an error, but it's still supported for backward compatibility. We (Jupyter extension) are working on a fix to migrate workbench.editorAssociations to the new format. |
Groovy - makes sense. I'll do the best to pass on the secret sauce to my students. Ironically, I was just showing them the cool/better notebook interface in class, then they got home and couldn't get it to work because of this :) Such is living on the edge. |
@flipphillips Yeah, sorry about the snafu. Team is out for the week but we should have this fixed Monday. |
This should be fixed in the latest Jupyter extension build targeting VS Code Insiders: v2021.6.828932339. |
Closing as this should have been resolved with today's update. Please feel free to open a new issue if otherwise. Thanks everyone for your patience and for reporting a bug! |
Issue Type: Bug
It seems that the last update crashed Jupyter notebook extension.
It cannot find
jupyter.opennotebook
function/object presumably need for opening.ipynb
files in the Notebook Editor. Moreover, there seems to be some even deeper problem going in the core of the extension which probably cannot load some sort of a configuration file (or so it seems according to the console log in Developer Tools. Below I paste the javascript erorr trace generated in my particular case and raised atworkbench.desktop.main.js:62
:Extension version: 2021.6.819764235
VS Code version: Code - Insiders 1.57.0-insider (6bee0f2f3fea9df1ad45344f09b2a139c93b384f, 2021-05-07T08:17:31.522Z)
OS version: Linux x64 5.11.0-7614-generic
System Info
gpu_compositing: enabled
multiple_raster_threads: enabled_on
oop_rasterization: disabled_off
opengl: enabled_on
rasterization: disabled_software
skia_renderer: enabled_on
video_decode: disabled_software
vulkan: disabled_off
webgl: enabled
webgl2: enabled
The text was updated successfully, but these errors were encountered: