Skip to content

Fix verify function to ensure all groups must pass - #825

Merged
taozha2 merged 1 commit into
intel:mainfrom
jinyouzhi:10_ggemm_verify_all_groups
Sep 22, 2026
Merged

taozha2 merged 1 commit into
intel:mainfrom
jinyouzhi:10_ggemm_verify_all_groups

Conversation

@jinyouzhi

Copy link
Copy Markdown

Description

For example 10th 10_bmg_grouped_gemm_mixed_dtype, the current verify function's pass condition is at least one group pass.
Change the pass condition as all groups need pass.

Type

  • Bug - [ ] Feature - [ ] Performance - [ ] Refactor

Testing

  • Tests pass - [ ] Xe12 - [ ] Xe20

Performance

Metric Before After

References

Fixes #

Checklist

  • Copyright - [ ] Co-pilot Review - [ ] Deprecated APIs not used

@tdeng5
tdeng5 requested review from jiyang1011 and taozha2 June 16, 2026 03:23
@taozha2
taozha2 enabled auto-merge (squash) September 22, 2026 06:23
@taozha2
taozha2 force-pushed the 10_ggemm_verify_all_groups branch from 7472c01 to ca1190d Compare September 22, 2026 06:23
@taozha2
taozha2 merged commit 4bd66b3 into intel:main Sep 22, 2026
7 of 9 checks passed
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