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

add GitHub CI for Intel GPU #536

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

add GitHub CI for Intel GPU #536

wants to merge 6 commits into from

Conversation

faaany
Copy link
Collaborator

@faaany faaany commented Jan 22, 2025

Summary

This PR adds the dependencies for using liger-kernel on XPU and adds the workflow.yml.

Testing Done

Test done on local XPU machine. To make all tests pass, we need to first merge PR #535 first.

  • Hardware Type: Intel(R) Data Center GPU Max 1550
  • run make test to ensure correctness
  • run make checkstyle to ensure code style
  • run make test-convergence to ensure convergence

@faaany
Copy link
Collaborator Author

faaany commented Jan 22, 2025

@faaany
Copy link
Collaborator Author

faaany commented Jan 23, 2025

@ByronHsu
Copy link
Collaborator

ByronHsu commented Jan 24, 2025

Is this ready to merge? How to enable our project to find linux-max1550-gpu-8? Do we need to give you some api key?

@faaany
Copy link
Collaborator Author

faaany commented Jan 24, 2025

Is this ready to merge? How to enable our project to find linux-max1550-gpu-8? Do we need to give you some api key?

Yes, the code is ready to merge. But we still need to configure the self-hosted runner for Liger-Kernel Repo. Maybe @hebiao064 can help with that?

@faaany
Copy link
Collaborator Author

faaany commented Jan 24, 2025

We also need to merge this PR first #498 to make the CI green on XPU, since the issue on triton-xpu might take some time to get fixed (see intel/intel-xpu-backend-for-triton#3237).

@ByronHsu
Copy link
Collaborator

Thank you! Just merged #498. And @hebiao064 will work with you on setting on self-hosted runner!

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.

2 participants