Skip to content

Conversation

@snnn
Copy link
Contributor

@snnn snnn commented Jul 28, 2025

fs-eire and others added 4 commits July 26, 2025 21:36
…ded but not constant. (#25544)

### Description
<!-- Describe your changes. -->

In DynamicQuantizeMatMul KleidiAI-specific prepacking logic, handle case
where B zero point input is provided but not constant. In this case, we
should not prepack.

Add some unit tests that test the prepacking code path.

Add check for ARM SME instructions in DynamicQuantizeMatMul before
calling `MlasDynamicQGemmBatch()` and associated functions.

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

Follow up to #25187
### Description
<!-- Describe your changes. -->
Refactor to split out classes and make things easier to find. 

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Cleanup
…ml (#25552)

### Description
Yesterday I updated the machine images. Now they already have python
preinstalled. We don't need to do this anymore.
Remove the steps to avoid conflicts.
Also, refactor the yaml file a little bit. Refactors templates to use
parameterized Python versions instead of matrix strategy.
@snnn
Copy link
Contributor Author

snnn commented Jul 28, 2025

There are some test failures in the Nuget Test pipelines. They were caused by recent moves of our pipelines and build machines. I will address them later this week.

@snnn snnn merged commit f80697a into main Jul 28, 2025
192 of 212 checks passed
@snnn snnn deleted the users/snnn/rel-1.23.0 branch July 28, 2025 21:07
@snnn snnn restored the users/snnn/rel-1.23.0 branch July 28, 2025 23:29
@snnn
Copy link
Contributor Author

snnn commented Jul 28, 2025

I am sorry. This PR targeted the wrong branch. I should set it to rel-1.23.0. I will create a new PR for this.

@jywu-msft
Copy link
Member

how should we fix this? should we revert this commit?

@fs-eire
Copy link
Contributor

fs-eire commented Jul 29, 2025

how should we fix this? should we revert this commit?

It’s OK. If you check the commit history, this commit contains no code change, so there is no need to revert anything. We need another PR CP to the release branch.

sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
- **DynamicQuantizeMatMul - handle case where B zero point input is
provided but not constant. (microsoft#25544)**
- **Refactor plugin EP support (microsoft#25541)**
- **Remove the python installation steps from
win-qnn-arm64-ci-pipeline.yml (microsoft#25552)**
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.

6 participants