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

Support running RayFed job in Ray client mode. #173

Merged
merged 25 commits into from
Sep 14, 2023
Merged

Conversation

jovany-wang
Copy link
Collaborator

Close #75

@jovany-wang
Copy link
Collaborator Author

TESTS should be added.

@jovany-wang jovany-wang requested review from zhouaihui, fengsp and NKcqx and removed request for zhouaihui August 15, 2023 09:52
Copy link
Collaborator

@fengsp fengsp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

fed/api.py Outdated Show resolved Hide resolved
@jovany-wang
Copy link
Collaborator Author

Tests was added.

@jovany-wang jovany-wang merged commit 7f1348a into main Sep 14, 2023
13 checks passed
@jovany-wang jovany-wang deleted the client_mode branch September 14, 2023 02:27
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

Successfully merging this pull request may close these issues.

[Feature] Support running RayFed job in Ray client mode.
2 participants