Skip to content

add dequantize and quantize back to contrib ops#1712

Merged
askhade merged 2 commits intomasterfrom
askhade/add_qops_back
Sep 6, 2019
Merged

add dequantize and quantize back to contrib ops#1712
askhade merged 2 commits intomasterfrom
askhade/add_qops_back

Conversation

@askhade
Copy link
Copy Markdown
Contributor

@askhade askhade commented Aug 28, 2019

Description: Adding DequantizeLinear and QuantizeLinear ops back to contrib ops for backward compatibility

Motivation and Context

  • When these 2 ops were promoted to onnx domain they were removed from contrib ops because they were fairly new and ORT did not have support for quantization earlier so there were no models out there using this op. However these 2 ops in particular were used by winml's weight packing script and therefore winml had produced models using the contrib op version. So need to add these back.

@askhade askhade requested a review from pranavsharma August 28, 2019 22:35
@askhade askhade requested a review from a team as a code owner August 28, 2019 22:35
@askhade askhade merged commit b2a2326 into master Sep 6, 2019
askhade added a commit that referenced this pull request Sep 12, 2019
* add dequantize and quantize back to contrib ops (#1712)

* fix build failures after cherry pick

* build fixes

* fix GPU build failures

* remove int8 kernel for DequatizeLinear

* fix contrib op test for dequantize linear
@askhade askhade deleted the askhade/add_qops_back branch September 30, 2020 23:20
yuslepukhin pushed a commit that referenced this pull request Mar 17, 2026
## Describe your changes

## 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