Skip to content
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

Enable armv8 CI #195

Merged
merged 3 commits into from
Dec 21, 2024
Merged

Enable armv8 CI #195

merged 3 commits into from
Dec 21, 2024

Conversation

digantdesai
Copy link
Collaborator

No description provided.

@digantdesai digantdesai force-pushed the test_armv8_ci branch 21 times, most recently from a96b925 to 00f231b Compare December 19, 2024 02:39
This does not exist for Darwin + Arm64.
TODO: Enable this selectively when possible.
@digantdesai digantdesai force-pushed the test_armv8_ci branch 6 times, most recently from bba2bfe to d59e8a9 Compare December 20, 2024 21:00
@digantdesai digantdesai marked this pull request as ready for review December 20, 2024 21:01
@digantdesai digantdesai requested a review from ptillet as a code owner December 20, 2024 21:01
@digantdesai digantdesai changed the title [not for merge] test armv8 CI Enable armv8 CI Dec 20, 2024
Copy link
Collaborator

@minjang minjang left a comment

Choose a reason for hiding this comment

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

Let's try!

@@ -757,7 +757,7 @@ def get_git_commit_hash(length=8):
"pytest-forked",
"pytest-xdist",
"scipy>=1.7.1",
"llnl-hatchet",
# "llnl-hatchet", # TODO: Re-enable this, not available on macos-arm64
Copy link
Collaborator

Choose a reason for hiding this comment

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

Will it be okay in other platforms? It's okay as you marked as TODO:

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is mainly used in the proton profiler. I guess we can commit it with a TODO, and revisit this later.

@digantdesai
Copy link
Collaborator Author

Ah still failing, let me fix it.

- Re-enable llnl-hatchet pip package installation
- look into faster triton install
- enable bf16 tests
- enable openmp
@digantdesai digantdesai merged commit 4e4e6b8 into main Dec 21, 2024
3 checks passed
@@ -12,6 +12,7 @@
from triton.backends.driver import DriverBase
from triton.backends.compiler import GPUTarget

from pathlib import Path
Copy link
Collaborator

Choose a reason for hiding this comment

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

hm is this import getting used?

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.

3 participants