Skip to content

[OpenVINO-EP] Update hardware branding of VAD-R as VAD-M - #1552

Merged
George Wu (jywu-mysoft) merged 1 commit into
microsoft:masterfrom
intel:manohar/vad-m_branding
Aug 5, 2019
Merged

[OpenVINO-EP] Update hardware branding of VAD-R as VAD-M#1552
George Wu (jywu-mysoft) merged 1 commit into
microsoft:masterfrom
intel:manohar/vad-m_branding

Conversation

@smkarlap

Copy link
Copy Markdown
Contributor

Description:
Replaces all occurrences of VAD-R/VAD_R with VAD-M/VAD_M.

Motivation and Context

  • Aligns with the Intel's official hardware branding.

Replaces all occurrences of VAD-R/VAD_R with VAD-M/VAD_M.
Aligns with the official hardware branding.
@jywu-mysoft

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 21 pipeline(s), but failed to run 1 pipeline(s).

@jywu-mysoft
George Wu (jywu-mysoft) merged commit 05bbb30 into microsoft:master Aug 5, 2019
@smkarlap
S. Manohar Karlapalem (smkarlap) deleted the manohar/vad-m_branding branch August 6, 2019 21:10
Dmitri Smirnov (yuslepukhin) pushed a commit that referenced this pull request Mar 17, 2026
…nn ep (#1565)

## Describe your changes
`quant_preprocess` was originally set to `False` by default because I
thought it might not be compatible with QNN EP. However, it is needed
for most models for the quantization to work properly:
- The quantizer needs a shape inferred model to quantize some tensors.
- Model quality is bad if constants are not made into initializers (see
#1552).

## Checklist before requesting a review
- [ ] Add unit tests for this change.
- [ ] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [ ] Lint and apply fixes to your code by running `lintrunner -a`
- [ ] Is this a user-facing change? If yes, give a description of this
change to be included in the release notes.
- [ ] Is this PR including examples changes? If yes, please remember to
update [example
documentation](https://github.com/microsoft/Olive/blob/main/docs/source/examples.md)
in a follow-up PR.

## (Optional) Issue link
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.

2 participants