-
Notifications
You must be signed in to change notification settings - Fork 40
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
fix: Remove psutil dependency #3439
base: main
Are you sure you want to change the base?
Conversation
@hpohekar I've also tested this approach. Does it work in Windows? |
@mkundu1 Right, it was not working in Windows but we have got 2 solutions, both of them are working fine. Personally I think 1st is better. Solution 1:
Solution 2:
I have checked it locally, tests are passing in Windows now. We have also added a separate test for standalone mode. |
Tested it, looks fine. Thanks. |
@hpohekar watchdog_exec also uses psutil. |
closes #3423
_pid_exists
in replacement ofpsutil.pid_exists()