Skip to content

introduce initial version of CI description for oneDAL #223

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

Merged
merged 10 commits into from
Apr 29, 2025

Conversation

napetrov
Copy link
Collaborator

This PR introducing CI description for oneDAL

@napetrov
Copy link
Collaborator Author

@theComputeKid @homksei @ahuber21 @ethanglaser @icfaust @rakshithgb-fujitsu @keeranroth -please be free to review/comment and ask questions

| Github | Intel GPU (ICX compiler) | Linux | 1 | Yes* | Pipelene configured for GPU valdiation on uxlfoundation GPU runners, currently disabled due to issues with Tiber Cloud. ci.yml[https://github.com/uxlfoundation/oneDAL/blob/main/.github/workflows/ci.yml] |
| Github | ABI conformance | Linux | 1 | Yes | 2 Pipelines on Physical ARM systems Github hosted and ARM hosted. ci.yml[https://github.com/uxlfoundation/oneDAL/blob/main/.github/workflows/ci.yml] |
| Github | Docker validation | Linux | 1 | Yes | Validation of oneDAL development env docker file. docker-validation-ci.yml[https://github.com/uxlfoundation/oneDAL/blob/main/.github/workflows/docker-validation-ci.yml] |
| Github | CPU Nightly | Linux, Windwos | 2 | Yes | Nightly builds and broaer validation for oneDAL. nightly-build.yml[https://github.com/uxlfoundation/oneDAL/blob/main/.github/workflows/nightly-build.yml] |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was some talk on separating the Nightly CI from those needed for PR acceptance: #218 (comment)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting - this puts whole another question on separation between PR level checks, CI level and scheduled tasks

Copy link
Contributor

@ethanglaser ethanglaser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Initial review, mostly for grammatical/typos

napetrov and others added 2 commits April 22, 2025 15:04
Co-authored-by: ethanglaser <42726565+ethanglaser@users.noreply.github.com>
Co-authored-by: Andreas Huber <9201869+ahuber21@users.noreply.github.com>
@napetrov napetrov requested review from rodburns and aahrun April 22, 2025 22:59
Copy link
Contributor

@ethanglaser ethanglaser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, very comprehensive

Co-authored-by: ethanglaser <42726565+ethanglaser@users.noreply.github.com>
@rodburns rodburns merged commit 903f01e into uxlfoundation:main Apr 29, 2025
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.

5 participants