Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Add option to limit CPU use to avoid the noisy fan (on Macbook pro) #113

Open
universemaster opened this issue Feb 12, 2023 · 3 comments
Open

Comments

@universemaster
Copy link

  • Is there a possibility of an option to limit CPU use?

    • On a 2014 Macbook pro when I use vale with the vscode extension, vale briefly spikes to above 200% fan use while I type, which turns on the noisy fan and makes visual updates (markdown preview etc) lag a bit.

    • I wonder if there could be an option to limit the CPU usage of vale?

    • I have very very many rules, which I'm reluctant to trim, because apart from the fan noise, I find these many rules a net benefit.

  • Another idea (on the vscode extension side) would be to "debounce" so that rather than running immediately on each key-press, vale is fired only after a pause of say 5-10 seconds? Again this could be configurable?

  • Finally, again on the vscode extension side, I suspect vscode is sending the entire file to vale on each key press? Would it be realistic to send only the part (paragraph?) that has changed. I've notice the CPU/noise problem becomes much worse, the longer the document?

@nschonni
Copy link

Sounds like most of this is for https://github.com/errata-ai/vale-vscode

@universemaster
Copy link
Author

Sounds like most of this is for https://github.com/errata-ai/vale-vscode

OK, do you really suggest I create an issue there, too? Seems a bit messy to create two? I was under the impression errata-ai was quite a small organisation?

@jdkato jdkato transferred this issue from errata-ai/vale Feb 13, 2023
@jdkato
Copy link
Member

jdkato commented Feb 13, 2023

Finally, again on the vscode extension side, I suspect vscode is sending the entire file to vale on each key press? Would it be realistic to send only the part (paragraph?) that has changed. I've notice the CPU/noise problem becomes much worse, the longer the document?

This was an option in a previous version:

"vale.server.lintContext": {

It should probably be re-introduced since it wasn't really dependent on Vale Server.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants