-
Notifications
You must be signed in to change notification settings - Fork 14.1k
CANN: Add Ascend CANN build ci #10217
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
Conversation
fc47d4f to
5ecac00
Compare
hipudding
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need use correct cmake command.
.github/workflows/build.yml
Outdated
| container: | ||
| image: ascendai/cann:openeuler-${{ matrix.cann }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only runs on openeuler?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first openeuler, and then ubuntu
|
Please use x86 image for build test. |
|
I think it's better triggered by 'Ascend NPU' label, I don't want it block others' PR. |
.github/workflows/build.yml
Outdated
| build: ['Release'] | ||
| device: ['ascend910b3'] | ||
| container: | ||
| image: ascendai/cann:${{ matrix.cann }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to format
Co-authored-by: Yuanhao Ji <jiyuanhao@apache.org>
|
@slaren Could you please review this PR? Thanks. This PR add build check for Ascend NPU, Only enable when "Ascend NPU" label set. |
* CANN: Add Ascend CANN build ci * Update build.yml * Modify cann image version * Update build.yml * Change to run on x86 system * Update build.yml * Update build.yml * Modify format error * Update build.yml * Add 'Ascend NPU' label restrictions * Exclude non PR event Co-authored-by: Yuanhao Ji <jiyuanhao@apache.org> * Update build.yml --------- Co-authored-by: Yuanhao Ji <jiyuanhao@apache.org>
* CANN: Add Ascend CANN build ci * Update build.yml * Modify cann image version * Update build.yml * Change to run on x86 system * Update build.yml * Update build.yml * Modify format error * Update build.yml * Add 'Ascend NPU' label restrictions * Exclude non PR event Co-authored-by: Yuanhao Ji <jiyuanhao@apache.org> * Update build.yml --------- Co-authored-by: Yuanhao Ji <jiyuanhao@apache.org>
No description provided.