-
Notifications
You must be signed in to change notification settings - Fork 148
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
Several inconsistencies found, trying to configure pylsp #1048
Comments
You do not have this extension installed (or enabled, or both). This is coming from JupyteLab 4.0+ and is non functional without this extension (or another one). This extension replaces |
Personally I am not happy that upstream includes |
Feel welcome to submit PR replacing |
Ugh, that one escaped me, I had jupyter-lsp, and jupyter-lsp-python, but not that one. Ironically, I wrote it correct in my old blog post, but STILL overlooked it now. :( Will send a PR. |
Hm, now i don't have ANY language server settings shown anymore.. because I filled something into the experimental settings? |
still have some stuff disabled, don't understand difference between @jupyterlab/lsp-extension (disabled) and @jupyter-lsp/jupyterlab-lsp (enabled). still no settings there, hmpf. |
Can you try choosing and reopening settings tab? There is a known issue with some settings categories not showing up when loading for the first time with Settings editor open. If it suits not help, is there anything in the browser console? |
Yes, this one suggests that you may have an invalid setting in jupyterlab-lsp/python_packages/jupyter_lsp/jupyter_lsp/specs/config/pylsp.schema.json Lines 5 to 14 in 5afe3b4
Can you check/clear your settings? These are stored on disk in a path indicated by |
Wait, you should not need to edit files on disk. You should be able to access the panel in the JSON Settings Editor |
Thank you, this is a bug. It should be there even if validation fails. |
What about editing the file on disk? |
yes, i removed my settings folder for jupyterlab-lsp and now the entry appears in the settings UI, thanks! |
Maybe i should open another DOCS issue about this, but I can't find on how to activate notebook formatting (for example isort or yapf) using pylsp in the notebook? I only find advice on how to run pylsp in the terminal... |
This project's main README describes installing an example server like so, using the Advanced Settings Editor:
But when trying to use the settings for Language Servers, since recently called "Language Servers (Experimental)", it seems one first needs to create a new language server with the right server name and manually activate it with a switch:
and then opening the JSON Settings Editor, some of the JSON keywords seem to not match above given instructions:
i.e.:
language_servers
is nowlanguageServers
pyls(p).serverSettings
is nowpyls(p).configuration
(indicating the recommended replacement of pyls with pylsp).
Note, that this is just an observation of inconsistencies, while I tried to find out how to get a python lsp server running, which I haven't been able to do so far. Something has changed as I was able to do to so before, even wrote a medium post about it...
Another observation: Anytime I added a property using the GUI, the terminal log said:
The text was updated successfully, but these errors were encountered: