-
Notifications
You must be signed in to change notification settings - Fork 4
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
[CI] Improve workflows #23
base: main
Are you sure you want to change the base?
Conversation
“ create an arm64 runner in your organization/enterprise, and select the “Ubuntu 24.04 by Arm Limited” partner image. ” Currently restricted for use in runners |
arm runers are available! |
.github/workflows/deepspeed.yaml
Outdated
deepspeed-ut: | ||
if: ${{ github.repository_owner == 'Ascend' }} | ||
name: Run unit tests with DeepSpeed | ||
runs-on: self-hosted |
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.
runs-on: Need to add specified labels as before
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.
fixed please help trigger ci again
@xuedinge233 please add a label named 'Ascend NPU', so we can trigger the llamacpp workflow |
mkdir build | ||
cd build | ||
export ASCEND_TOOLKIT_HOME=/usr/local/Ascend/ascend-toolkit/latest |
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.
It's not required to export this, but LD_LIBRARY_PATH must include ${ASCEND_TOOLKIT_HOME}/lib64, although this path already exists in LD_LIBRARY_PATH, this may seem strange, but it's necessary.
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.
added some logs, please trigger ci and let's see what the logs say
cc: @FFFrog @xuedinge233