Skip to content

Add functionality to FusingInfo class to preserve quantization config#1426

Merged
kkawa14 merged 30 commits intoSonySemiconductorSolutions:mainfrom
kkawa14:apply_quant_info_to_fusinginfo
May 14, 2025
Merged

Add functionality to FusingInfo class to preserve quantization config#1426
kkawa14 merged 30 commits intoSonySemiconductorSolutions:mainfrom
kkawa14:apply_quant_info_to_fusinginfo

Conversation

@kkawa14
Copy link
Copy Markdown
Contributor

@kkawa14 kkawa14 commented Apr 25, 2025

Pull Request Description:

Add functionality to the FusingInfo class to preserving fusion quantization configs.
Add functionality to the FQC to get fusion quantization config from the Fusing class.
Add unittests for the Fusing class.

Checklist before requesting a review:

  • I set the appropriate labels on the pull request.
  • I have added/updated the release note draft (if necessary).
  • I have updated the documentation to reflect my changes (if necessary).
  • All function and files are well documented.
  • All function and classes have type hints.
  • There is a licenses in all file.
  • The function and variable names are informative.
  • I have checked for code duplications.
  • I have added new unittest (if necessary).

Abdullah4345
Abdullah4345 previously approved these changes Apr 28, 2025
@ofirgo ofirgo dismissed Abdullah4345’s stale review April 29, 2025 09:03

no content to the review

Copy link
Copy Markdown
Contributor

@reuvenperetz reuvenperetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Left some small comments...

@kkawa14 kkawa14 requested a review from reuvenperetz May 8, 2025 05:23
@kkawa14 kkawa14 merged commit 056da80 into SonySemiconductorSolutions:main May 14, 2025
28 checks passed
@kkawa14 kkawa14 deleted the apply_quant_info_to_fusinginfo branch May 23, 2025 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants