Skip to content

Draft: Add Unified Attention UTs #374

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Draft: Add Unified Attention UTs #374

wants to merge 7 commits into from

Conversation

kzawora-intel
Copy link
Collaborator

First draft, probably completely broken now.

kzawora-intel and others added 2 commits October 9, 2025 17:44
@kzawora-intel @tthaddey
Add non-unified attention UTs
114936a 
Co-authored-by: Tomasz Thaddey <[email protected]>
Signed-off-by: Konrad Zawora <[email protected]>
@kzawora-intel
Add unified attention UTs
72f5112 
Signed-off-by: Konrad Zawora <[email protected]>
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 9, 2025

🚧 CI Blocked

The main CI workflow was not started for the following reason:

This is a Draft PR. Please mark it as 'Ready for Review' to trigger the CI.

@kzawora-intel
fix api mismatches
8cfe6d3 
Signed-off-by: Konrad Zawora <[email protected]>
@github-actions GitHub Actions
Copy link

🚧 CI Blocked

The main CI workflow was not started for the following reason:

This is a Draft PR. Please mark it as 'Ready for Review' to trigger the CI.

@kzawora-intel
stuff
3742ccd 
Signed-off-by: Konrad Zawora <[email protected]>
@kzawora-intel
add cosine sim tests
67fceea 
Signed-off-by: Konrad Zawora <[email protected]>
@github-actions GitHub Actions
Copy link

🚧 CI Blocked

The main CI workflow was not started for the following reason:

This is a Draft PR. Please mark it as 'Ready for Review' to trigger the CI.

@github-actions GitHub Actions
Copy link

🚧 CI Blocked

The main CI workflow was not started for the following reason:

This is a Draft PR. Please mark it as 'Ready for Review' to trigger the CI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xuechendi xuechendi Awaiting requested review from xuechendi xuechendi will be requested when the pull request is marked ready for review xuechendi is a code owner

@mswiniarsk mswiniarsk Awaiting requested review from mswiniarsk mswiniarsk will be requested when the pull request is marked ready for review mswiniarsk is a code owner

@adobrzyn adobrzyn Awaiting requested review from adobrzyn adobrzyn will be requested when the pull request is marked ready for review adobrzyn is a code owner

@mgawarkiewicz-intel mgawarkiewicz-intel Awaiting requested review from mgawarkiewicz-intel mgawarkiewicz-intel will be requested when the pull request is marked ready for review mgawarkiewicz-intel is a code owner

@vivekgoe vivekgoe Awaiting requested review from vivekgoe vivekgoe will be requested when the pull request is marked ready for review vivekgoe is a code owner

@afierka-intel afierka-intel Awaiting requested review from afierka-intel afierka-intel will be requested when the pull request is marked ready for review afierka-intel is a code owner

@michalkuligowski michalkuligowski Awaiting requested review from michalkuligowski michalkuligowski will be requested when the pull request is marked ready for review michalkuligowski is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kzawora-intel