-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
[CI] Test federated learning plugin in the CI #8325
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Nevermind,
I have no idea why this is happening. |
@rongou Hi, is this the last PR for federated learning planned for 1.7? |
@hcho3 I couldn't reproduce the error with a bare metal machine, maybe it's a docker-specific constraint on the network? Some additional notes:
|
@trivialfis yes this should be the last PR for 1.7. #8316 is a nice to have. As for |
I fixed #8325 (comment) by installing gRPC in a Conda environment. My guess is that I accidentally overwrote some headers in the system when I installed gRPC globally. |
This reverts commit 90462c0.
@rongou Can you review the changes I made to gtest? I had to randomize the server port, because reusing the same port twice often resulted into a connection failure. |
GTest is finally fixed 🎉 |
Close #8301
Close #8296
Notes: