How to show the result of offline CI for SYCL backend #6399
NeoZhangJianyu
started this conversation in
General
Replies: 1 comment 18 replies
-
We successfully deployed an approach with self-hosted github runner for CUDA, maybe the same can be done for SYCL ? |
Beta Was this translation helpful? Give feedback.
18 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Background:
Current CI of SYCL backend is only building code, without running unit test.
Run the unit test need Intel GPU, which is not provided by github runner.
It depends on developer running local unit test before create PR.
For many reasons, the unit test pass rate is easy to be broken from 100%.
That leads to the result of OPs have obvious error, performance drop.
That impacts the whole SYCL backend quality and user experience.
Solution:
Offline CI for SYCL backend:
1. github doesn't recommend to setup self-hosted runner for public repo, due to security issue.
2. I run the unit test/performance for SYCL backend in my personal computer with Intel iGPU.
I choose the existed commits which change SYCL backend.
I could know the quality and performance of every commit.
Question:
How to show the result of offline CI for SYCL backend?
I hope to publish the result of every key commit.
It will help developer to improve the quality and user to choose the release with best quality and performance.
Alternative:
a. Update the summary of unit test result of SYCL backend in README-sycl.md
b. Insert the unit test pass rate in the title of merged PR.
c. Update the summary of test result in another public repo MD file and insert the link in README-sycl.md. So user is easy to read it to check the latest SYCL backend test result.
I choose c.
How about your idea?
@ggerganov @slaren @airMeng
Beta Was this translation helpful? Give feedback.
All reactions