-
Notifications
You must be signed in to change notification settings - Fork 12.2k
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
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 <[email protected]>
@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 <[email protected]> * Update build.yml --------- Co-authored-by: Yuanhao Ji <[email protected]>
* 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 <[email protected]> * Update build.yml --------- Co-authored-by: Yuanhao Ji <[email protected]>
* 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 <[email protected]> * Update build.yml --------- Co-authored-by: Yuanhao Ji <[email protected]>
* 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 <[email protected]> * Update build.yml --------- Co-authored-by: Yuanhao Ji <[email protected]>
No description provided.