Skip to content

Handle quantization preserving with 2 inputs#1421

Merged
elad-c merged 3 commits intomainfrom
fix_q_preserve_for_two_input_nodes
Apr 22, 2025
Merged

Handle quantization preserving with 2 inputs#1421
elad-c merged 3 commits intomainfrom
fix_q_preserve_for_two_input_nodes

Conversation

@elad-c
Copy link
Copy Markdown
Contributor

@elad-c elad-c commented Apr 22, 2025

Pull Request Description:

Handle quantization preserving nodes that have more than 1 input, or come after an unquantized node. For these nodes, quantization preserving is disabled.

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).

…come after an unquantized node. For these nodes, quantization preserving is disabled.
@elad-c elad-c requested a review from irenaby April 22, 2025 09:09
@elad-c elad-c merged commit 0dbb760 into main Apr 22, 2025
30 checks passed
@elad-c elad-c deleted the fix_q_preserve_for_two_input_nodes branch April 22, 2025 13:38
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.

2 participants