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

Ray Train integration with LLaMA-Factory #6542

Merged
merged 4 commits into from
Jan 8, 2025

Conversation

erictang000
Copy link
Contributor

@erictang000 erictang000 commented Jan 6, 2025

What does this PR do?

Fixes #6543

Before submitting

@hiyouga hiyouga added the pending This problem is yet to be addressed label Jan 7, 2025
@hiyouga hiyouga self-requested a review January 7, 2025 03:14
kouroshHakha and others added 2 commits January 7, 2025 08:55
Signed-off-by: Kourosh Hakhamaneshi <kourosh@anyscale.com>
Copy link
Owner

@hiyouga hiyouga left a comment

Choose a reason for hiding this comment

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

LGTM

@hiyouga hiyouga force-pushed the et/ray-integration branch from b2709c1 to d8cac6f Compare January 7, 2025 09:39
@kouroshHakha
Copy link
Contributor

Nice @hiyouga. Thanks for taking quick review action on this.

@hiyouga hiyouga merged commit d23a988 into hiyouga:main Jan 8, 2025
12 checks passed
@hiyouga hiyouga added solved This problem has been already solved and removed pending This problem is yet to be addressed labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved This problem has been already solved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Ray Train integration with LLaMA-Factory
3 participants