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

Autosave option enabled cause few simultaneous analysis processes. #31

Open
ArtsiomCh opened this issue Mar 12, 2020 · 6 comments
Open

Comments

@ArtsiomCh
Copy link

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.

@deepcodeg
Copy link
Contributor

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

@Arvi3d
Copy link
Contributor

Arvi3d commented Apr 27, 2020

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

@JustFly1984
Copy link

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.

@Arvi3d
Copy link
Contributor

Arvi3d commented Jun 12, 2020

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

@JustFly1984
Copy link

JustFly1984 commented Jun 14, 2020

@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
Copy link

JustFly1984 commented Jun 23, 2020

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

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

No branches or pull requests

4 participants