Skip to content
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

1.17.2, no ending processing incremental #437

Closed
Fred-Vatin opened this issue Mar 2, 2021 · 35 comments
Closed

1.17.2, no ending processing incremental #437

Fred-Vatin opened this issue Mar 2, 2021 · 35 comments
Labels
bug Something isn't working

Comments

@Fred-Vatin
Copy link

VS Code version OS Extension version
1.53.2 Win 10 x64 pro (french) 18363.1379 1.17.2

Steps to reproduce

  1. Open a lua file that used to work with v1.17.0
  2. Put the cursor on any function or variable
  3. Go to definition or references fails
  4. Status bar shows this processing that never ends
    image

Results

Go to definition or references fails

Environment (please complete the following information):

  • Is WSL remote: NO

##Workaround
Revert to v1.17 solves the issue.

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

@sumneko sumneko added the bug Something isn't working label Mar 3, 2021
@Ketho
Copy link

Ketho commented Mar 3, 2021

For me the status bar stays stuck at "Processing hover", at least with emmylua files loaded.
But I sometimes also get stuck on "Processing incremental semantic token" before I hover over anything.

lua.language.server.1.17.2.mp4

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

76f1b37
Please try to apply this commit manually and see if the issue is resolved

@Ketho
Copy link

Ketho commented Mar 3, 2021

I edited the file C:\Users\Ketho\.vscode\extensions\sumneko.lua-1.17.2\server\script\files.lua and manually removed those lines but it didn't fix the issue after restarting :(

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

cc6ec7e
Please try this

@Ketho
Copy link

Ketho commented Mar 3, 2021

Nope, still stuck on hover

image

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Please show me the new log

@Ketho
Copy link

Ketho commented Mar 3, 2021

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

From the log, it seems that you haven't done any operation. Please make sure that you sent the correct log.
The main problem is that I cannot reproduce this issue. Can you provide a demo project?

@Ketho
Copy link

Ketho commented Mar 3, 2021

I was sure I tried to hover over one of my variables. The hover does work for functions though.
Could you try this demo project? I tried to make it as small as possible.
test.zip

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

I can reproduce it with your demo project, many thanks!

@Fred-Vatin
Copy link
Author

Please show me your log, see https://github.com/sumneko/lua-language-server/wiki/Default-log-path

Here is the log from v1.17.2 when opening any lua file like this one.

sumneko added a commit that referenced this issue Mar 3, 2021
@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Please try this commit

@Fred-Vatin
Copy link
Author

Please try this commit

This seems good ! 👍

@Fred-Vatin
Copy link
Author

But still the same issue on 1.17.3
image

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

I have tested #437 (comment) in 1.17.3 and it works well.
Is that works well for you?
Could you provide a demo project?

@Fred-Vatin
Copy link
Author

Fred-Vatin commented Mar 3, 2021

I have tested #437 (comment) in 1.17.3 and it works well.
Is that works well for you?
Could you provide a demo project?

Still the same lua script I provided before but now it is stuck at
image
I tried to change the #437 (commit) to no avail.

Here is the log with #437 (commit) applied.

Here is my settings for the extensions.

	// # EXTENSION Lua
	"Lua.completion.callSnippet": "Both",
	"Lua.diagnostics.globals": ["gg"],
	"Lua.runtime.version": "Lua 5.2",
	"Lua.diagnostics.severity": {
		"unused-function": "Hint",
		"unused-local": "Warning"
	},
	"Lua.diagnostics.workspaceDelay": 1000,
	"Lua.hint.enable": true,
	"Lua.hint.setType": true,

1.17.0 is blazingly fast and bug free for me with these settings.

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Did you mean you have tried this project https://github.com/sumneko/lua-language-server/files/6074215/test.zip in 1.17.3, and it stucked when hover model at test.lua?

@Fred-Vatin
Copy link
Author

Fred-Vatin commented Mar 3, 2021

Did you mean you have tried this project https://github.com/sumneko/lua-language-server/files/6074215/test.zip in 1.17.3, and it stucked when hover model at test.lua?

I’m sorry, I provided the wrong link. Here it is.

It happens as soon as I load any lua file in VSC, even doing nothing. I updated and corrected my previous comment to add more details if it helps.

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Im a little confused.
So at your side, the issue is gone in 1.17.2 with applying commit b06ba33, but come back in 1.17.3, am I right?

@Fred-Vatin
Copy link
Author

Fred-Vatin commented Mar 3, 2021

Im a little confused.
So at your side, the issue is gone in 1.17.2 with applying commit b06ba33, but come back in 1.17.3, am I right?

It seemed yes. I replaced the guide.lua in the 1.17.2 with the one provided in #437 (commit). Then all seemed fine.

Then I updated to 1.17.3, took car to delete previous version in %homepath%.vscode\extensions and still buggy.

Did the log from 1.17.3 help ?

@Fred-Vatin
Copy link
Author

The guide.lua I downloaded then was this one. Is this the one you wanted me to test, right ?

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Yes.

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Eventually I found that there are 2 bugs, one of which I have fixed in 1.17.3, and the other bug only occurs when opening a single file.
This is why I have been unable to reproduce this problem before, because another person provided a folder containing multiple files, and I always opened the entire folder directly, so everything was normal. I haven't found that you only opened a single file. This is indeed reflected in the log, but I didn't notice it.
Now I can reproduce your problem, but it is late at night here, and I will fix it tomorrow.

@Fred-Vatin
Copy link
Author

Eventually I found that there are 2 bugs, one of which I have fixed in 1.17.3, and the other bug only occurs when opening a single file.
This is why I have been unable to reproduce this problem before, because another person provided a folder containing multiple files, and I always opened the entire folder directly, so everything was normal. I haven't found that you only opened a single file. This is indeed reflected in the log, but I didn't notice it.
Now I can reproduce your problem, but it is late at night here, and I will fix it tomorrow.

I’m glad you could reproduce the bug. Good night.

@Fred-Vatin
Copy link
Author

Im a little confused.
So at your side, the issue is gone in 1.17.2 with applying commit b06ba33, but come back in 1.17.3, am I right?

It seemed yes. I replaced the guide.lua in the 1.17.2 with the one provided in #437 (commit). Then all seemed fine.

Then I updated to 1.17.3, took car to delete previous version in %homepath%.vscode\extensions and still buggy.

Did the log from 1.17.3 help ?

@Fred-Vatin Fred-Vatin reopened this Mar 3, 2021
@Fred-Vatin
Copy link
Author

Fred-Vatin commented Mar 3, 2021

The guide.lua I downloaded then was this one. Is this the one you wanted me to test, right ?

(Oups, sorry for having closed the issue. Wrong button while I was writing a comment)

I did the test again with 1.17.2 replacing the guide.lua by the one above and failed to reproduce the success. I have the same issue with 1.17.2 with modified guide.lua. After restarted VSC, of course.
image

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

The guide.lua I downloaded then was this one. Is this the one you wanted me to test, right ?

(Oups, sorry for having closed the issue. Wrong button while I was writing a comment)

I did the test again with 1.17.2 replacing the guide.lua by the one above and failed to reproduce the success. I have the same issue with 1.17.2 with modified guide.lua. After restarted VSC, of course.
image

Open single file or open folder?

@Fred-Vatin
Copy link
Author

The guide.lua I downloaded then was this one. Is this the one you wanted me to test, right ?

(Oups, sorry for having closed the issue. Wrong button while I was writing a comment)
I did the test again with 1.17.2 replacing the guide.lua by the one above and failed to reproduce the success. I have the same issue with 1.17.2 with modified guide.lua. After restarted VSC, of course.
image

Open single file or open folder?

Single file

@Ketho
Copy link

Ketho commented Mar 3, 2021

Eventually I found that there are 2 bugs, one of which I have fixed in 1.17.3, and the other bug only occurs when opening a single file.

Yea there seem to be 2 separate bugs, I also encountered the issue with only opening a single file but I just assumed they were related... Sorry for that, 晚安~

@sumneko
Copy link
Collaborator

sumneko commented Mar 3, 2021

Eventually I found that there are 2 bugs, one of which I have fixed in 1.17.3, and the other bug only occurs when opening a single file.

Yea there seem to be 2 separate bugs, I also encountered the issue with only opening a single file but I just assumed they were related... Sorry for that, 晚安。

It’s okay, the performance of these two bugs is very similar, it is really difficult to distinguish

sumneko added a commit that referenced this issue Mar 4, 2021
@sumneko
Copy link
Collaborator

sumneko commented Mar 4, 2021

Please manually update script/plugin.lua and check if it works!

@Ketho
Copy link

Ketho commented Mar 4, 2021

Works for me! It no longer gets stuck on "Processing incremental semantic tokens" when opening a single Lua file.

@Fred-Vatin
Copy link
Author

Please manually update script/plugin.lua and check if it works!

I didn’t have to. 1.17.4 solved this issue for me.

@sumneko
Copy link
Collaborator

sumneko commented Mar 5, 2021

Seems fixed, closing

@sumneko sumneko closed this as completed Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants