-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Unknown file error on textDocument/hover #717
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
Comments
Hey @0x7CFE , what Operating System are you on? (I am seeing something similar in my attempt to use rust-analyzer with Visual Studio Code on Mac OS X. My first instinct is that its something vfs, which matches up with @matklad 's hypothesis over here: #773 (comment) ) |
Hi! I'm using Ubuntu Linux 16.04.5 and looks like this issue disappeared in recent versions of Rust Analyzer. Just in case, my directory hierarchy contains symlinks, maybe that affected the issue somehow. |
Closing, presumably fixed. |
Is there any way to suppress all or specific errors from the rust-analyzer vscode extension? 🙏 I have this error pop often hundreds of times a day (that's no hyperbole). It's painfully distracting. Each time it happens vscode plays the toast animation regardless of if I've cleared it or not. I tried v0.4.1367 just to make sure this isn't fixed in a later version (published 4 hours ago). VSCode won't provide a means of error suppression. Cant' say I agree with that; blacklisting repeat errors seems like a perfectly reasonable request, but c'est la vie. My only hope at holding onto sanity is stopping the error propagation. |
Much better would be to actually fix the error. For that, a minimal reproducible example would be the first step. |
@matklad It's possible there is something goofy with (all) of my systems, because a 'minimal repro' is pretty much any rust development, on either WSL2 or SSH to Ubuntu. If I have compiler errors and start moving the cursor around I often get these. If that's abnormal I can do some digging? I hear you on wanting to fix the root of the error. That a valiant desire. But, counterargument... I use I hope this doesn't come across as negative, I value the rust community and tooling immensely! This one bug has just been driving me a bit batty, like a splinter you can't get at. |
Yep, easy to do, just tabbed back to the project I have open and wiggled the mouse a bit. Link to project source is in there too: https://gist.github.com/AThilenius/f7ea00c9bf61720c9ab2680e103b40e4 Env: |
That is #13951 |
Automated crash reporting is a better solution for gathering errors than
user-pain.
Yeah, that’d be a better solution, but that runs into two problems:
* someone has to operate crash-report collecting software. I don’t think
either Rust project or VS Code have anything built-in for us to re-use?
* automated collection of use data in form of crash reports is a very
sensitive topic, so some amount of community-wide consensus building would
be required first.
I wonder if can make the error message more helpful here though? It should
at least teach the user where to look for panic stack-trace…
…On Sunday, 15 January 2023, Lukas Wirth ***@***.***> wrote:
If it happens on all projects its tough to make a minimal repro so dw :)
Can you check the logs the next time it happens (ideally it should show a
stack trace). Relevant log is Rust Analyzer Language Server
[image: image]
<https://user-images.githubusercontent.com/3757771/212570164-b8ddccf4-af7e-4c8a-a077-bb8a77ea7d7e.png>
—
Reply to this email directly, view it on GitHub
<#717 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANB3M5WGYJTYGYCPPYWEG3WSRZ33ANCNFSM4GTQAABA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Another option would be to add a "report bug" button to the error message which opens a pre-filled new issue page in your browser but not submit it so you can still remove anything you don't like to share. That would significantly reduce friction. I myself skip reporting issues 9 out of 10 times because filling out an issue takes several minutes. |
We can't change the error message notification unfortunately, I've looked into that into the past and it happens too early in the language client implementation of VSCode. |
Looks like VSCode has some support for it: https://code.visualstudio.com/api/extension-guides/telemetry It also has guidelines on user consent and PII. Telemetry data will be more useful for this kind of stuff anyway, because you'll implicitly know if it's systemic or something with my specific setup. If you're hoping to stick with human-triggered dumps, then I think there are many solutions you could explore. Trap the crash somehow (dump to file and check for existence on reboot, or catch_unwind if it's feasible) then show a custom notification with a report button. My feature request there would be to have a 'Suppress Similar Errors' button as well, maybe in the crash dump dialog box. Edit: we've strayed way off topic from the original post, should I open a separate Issue for this? |
Uh oh!
There was an error while loading. Please reload this page.
tldr: When I load my project Rust Analyzer starts flooding the console with messages like these:
Please note, that the path
/home/korvin/work/Parity/substrate-node-template/runtime/src/utxo.rs
is valid and it's one of the files being edited. The same file may be found here https://github.com/paritytech/substrate-node-template/blob/utxo/runtime/src/utxo.rs.HEAD of the analyzer repo:
The text was updated successfully, but these errors were encountered: