New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixed a lack of processing of OP name sanitizing when the `-onimc` and `-osd` or `-oiqt` options are specified in combination. #445

Merged

PINTO0309 merged 2 commits into main from fix_onimc

Aug 8, 2023

Owner

PINTO0309 commented Aug 8, 2023

1. Content and background

onnx2tf.py
- Fixed a lack of processing of OP name sanitizing when the -onimc and -osd or -oiqt options are specified in combination.
- osnet_x1_0_fp_32_bs_1.onnx.zip
```
onnx2tf \
-i osnet_x1_0_fp_32_bs_1.onnx \
-onimc /conv2/conv2.0/Relu_output_0 \
-oiqt \
-qt per-tensor
```

2. Summary of corrections

3. Before/After (If there is an operating log that can be used as a reference)

4. Issue number (only if there is a related issue)

[OSNet] int8 tflite model - catastrophic accuracy degradation #444

PINTO0309 added 2 commits

August 8, 2023 22:32


          Fixed a lack of processing of OP name sanitizing when the -onimc an…

884c7ef

…d `-osd` or `-oiqt` options are specified in combination.


          Reduce frequency of name duplication after sanitizing. : -> __

141774f

PINTO0309 merged commit 32534d7 into main

PINTO0309 deleted the fix_onimc branch

August 8, 2023 16:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet