We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
There is two entry point of the parameter server instance, the one is Python /elasticdl/python/ps, another one is /elasticdl/go/cmd/elasticdl_ps.
It seems the Python code is useless, can we remove it from this repo? c.f.
elasticdl/elasticdl/python/master/master.py
Line 414 in 6b1a1b2
The text was updated successfully, but these errors were encountered:
Yes, we could remove the parameter server implementation with Python.
The parameter server implementation with Go has been used in production already. It's safe to remove Python codes.
Sorry, something went wrong.
wangkuiyi
QiJune
Successfully merging a pull request may close this issue.
There is two entry point of the parameter server instance, the one is Python /elasticdl/python/ps, another one is /elasticdl/go/cmd/elasticdl_ps.
It seems the Python code is useless, can we remove it from this repo?
c.f.
elasticdl/elasticdl/python/master/master.py
Line 414 in 6b1a1b2
The text was updated successfully, but these errors were encountered: