Skip to content

Modify FLN Activation Quantization Holder initialization in model.#1494

Closed
gouda-youichi wants to merge 4 commits intoSonySemiconductorSolutions:mainfrom
kkawa14:adding_fln_actq_holder_init_in_model
Closed

Modify FLN Activation Quantization Holder initialization in model.#1494
gouda-youichi wants to merge 4 commits intoSonySemiconductorSolutions:mainfrom
kkawa14:adding_fln_actq_holder_init_in_model

Conversation

@gouda-youichi
Copy link
Copy Markdown
Contributor

@gouda-youichi gouda-youichi commented Jul 9, 2025

Pull Request Description:

Modify FLN Activation Quantization Holder initialization in model.

  • Update model builder to retrieve inside FLN nodes quantization information (from FusingInfo) and use it to initialize the quantizer in the FLNActivationQuantizationHolder.
  • Resolve the mismatch between fuseinfo-related strings and method names.

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

kawakami-masaki0 and others added 4 commits July 4, 2025 10:44
* add test code for FLN

* modified the model

* add space

* minor corrections

* fix fw_info in test code

* add FLN_NO_QUANT test

* change 16bit for FLN

* Resolve test errors

* Revert "Resolve test errors"

This reverts commit 0e77645.

* Support for QuantizationConfig changes

* Added an argument to the GradientPTQConfig function.

* Change description format

* Revert "Change description format"

This reverts commit c83f4a4.

---------

Co-authored-by: kawakami-masaki0 <kawakami.masaki@jp.panasonic.com>
@KazunoriSumiya KazunoriSumiya deleted the adding_fln_actq_holder_init_in_model branch July 9, 2025 06:53
@KazunoriSumiya KazunoriSumiya restored the adding_fln_actq_holder_init_in_model branch July 9, 2025 07:48
@KazunoriSumiya KazunoriSumiya reopened this Jul 9, 2025
@gouda-youichi
Copy link
Copy Markdown
Contributor Author

gouda-youichi commented Jul 9, 2025

I accidentally deleted the source branch.
I restored it and reopened.

@gouda-youichi gouda-youichi changed the title Adding fln activation quantization holder init in model Modify FLN Activation Quantization Holder initialization in model. Jul 9, 2025
@ofirgo ofirgo self-requested a review July 10, 2025 08:03
@gouda-youichi gouda-youichi requested review from kawakami-masaki0 and removed request for kawakami-masaki0 July 15, 2025 03:05
@ofirgo ofirgo closed this Aug 4, 2025
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.

5 participants