-
Notifications
You must be signed in to change notification settings - Fork 487
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
BytePS requires root permission #9
Comments
We will make these paths configurable. Also, maybe use /tmp/ as default path, though /tmp/ is a dangerous location for other reasons.. |
Thank you for the report. I will keep you updated once this is fixed. @bobzhuyb Let me update the code. |
@ppwwyyxx Commit |
Thanks for the quick response! |
* wd: use Process * wd: add log * wd: fix bug * wd: check context * wd: add log * wd: use threading.Thread * wd: use deepcopy * wd: fix * wd: fix typo
Describe the bug
BytePS needs to create socket hard-coded at this path:
byteps/byteps/common/communicator.h
Lines 34 to 35 in b9811f7
As a result, BytePS cannot be used without root permission. Running BytePS without root permission gives:
The text was updated successfully, but these errors were encountered: