vscode-extension icon indicating copy to clipboard operation
vscode-extension copied to clipboard

Autosave option enabled cause few simultaneous analysis processes.

Open ArtsiomCh opened this issue 4 years ago • 6 comments

With Autosave enabled DeepCode analysis starts for every symbol typed/deleted. That cause at least 3 concurrent analysis processes working at the same time: image

Imho when new analysis started the older one should be cancelled.

PS Also probably not all files need to be re-uploaded, just edited one should.

ArtsiomCh avatar Mar 12 '20 12:03 ArtsiomCh

@ArtsiomCh thanks for the report, we are working on it. I hope we can get a version out pretty soon fixing this error!

deepcodeg avatar Mar 16 '20 09:03 deepcodeg

Hi @ArtsiomCh. We release a new major version, that should have fixed this issue

Arvi3d avatar Apr 27 '20 18:04 Arvi3d

hello, I still observing multiple processes in parallel, and some false negatives, which doesn't solved by ignore comments. I have 400+ false negatives in typescript/react project.

JustFly1984 avatar Jun 11 '20 22:06 JustFly1984

thank you @JustFly1984 I replied also in your issue #42

Arvi3d avatar Jun 12 '20 08:06 Arvi3d

@Arvi3d one more thing I have noticed - deepcode analysis starts then I'm running a build with claudia.js in my project, which removes old dist directory with rimraf before build and recreates it.

dist directory is in .gitignore files. I think it could be a good practice to ignore everything what is in .gitignore for performance.

JustFly1984 avatar Jun 14 '20 08:06 JustFly1984

@Arvi3d Also it runs the process on build for typescript project. I guess it should respect .gitignore for tracking changes to solve an issue

JustFly1984 avatar Jun 23 '20 21:06 JustFly1984