search

posit-dev / positron

Positron, a next-generation data science IDE
Other
1.79k stars 50 forks source link

Trigger code completion in R raised error notification #3835

Open ntluong95 opened 2 weeks ago

ntluong95 commented 2 weeks ago

Positron Version:

Positron Version: 2024.06.1 (system setup) build 2024.06.1-54 Code - OSS Version: 1.90.0 Commit: 5115c73e0670f39139e55eeedf0717b468a0658d Date: 2024-07-03T03:17:46.140Z Electron: 29.4.0 Chromium: 122.0.6261.156 Node.js: 20.9.0 V8: 12.2.281.27-electron.0 OS: Windows_NT x64 10.0.22621

Steps to reproduce the issue:

  1. Import a dataframe in R
  2. Trigger code completion by pressing Ctrl + Space

What did you expect to happen?

Show the list of functions and arguments normally without any error

Were there any error messages in the output or Developer Tools console?

image

lionel- commented 2 weeks ago

Can you consistently reproduce this error notification? If that's the case, could you share with us a reproducible example, for instance the data frame contents that causes this error?

It would also be helpful to get the logs after the error occurred, both for the LSP and for the Console. You can find the logs in the Ouput panel:

Screenshot 2024-07-03 at 10 44 59 Screenshot 2024-07-03 at 10 45 12 Screenshot 2024-07-03 at 10 45 46
juliasilge commented 1 week ago

We believe failures of this kind will be addressed by https://github.com/posit-dev/ark/pull/431.

Maybe not, actually!

yutannihilation commented 1 week ago

Thanks for catching, but, looking at the attached screenshot, I'm not 100% sure if this is the same problem as https://github.com/posit-dev/ark/pull/431; in the screenshot, mutate() shows the correct candidates of the arguments, so this probably means the language server did succeed to find the corresponding help page. Of course, it's possible that the error occurred on different topic than mutate().

(I wish the reporter can eventually find some reproducible example of this error.)

juliasilge commented 1 week ago

Ah, I see what you mean @yutannihilation, OK. We'll bump this back for triage.