Skip to content
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.

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

Added sanitizing of : and / just before first dummy inference #329

Merged
merged 2 commits into from
Apr 26, 2023

Conversation

PINTO0309
Copy link
Owner

@PINTO0309 PINTO0309 commented Apr 26, 2023

1. Content and background

  • onnx2tf.py

    • Added sanitizing of : and / just before first dummy inference
    • ScaleMul.onnx.zip
    • Before
      onnx2tf \
      -i ScaleMul.onnx \
      -kat onnx__Mul_0 onnx__Mul_1 \
      -cind "onnx::Mul_0" pos.npy \
      -cind "onnx::Mul_1" pos_scales.npy \
      -cotof
      
    • After
      onnx2tf \
      -i ScaleMul.onnx \
      -kat onnx__Mul_0 onnx__Mul_1 \
      -cind "onnx__Mul_0" pos.npy \
      -cind "onnx__Mul_1" pos_scales.npy \
      -cotof
      
  • onnx
    image

  • result
    image

  • tflite
    image

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)

@PINTO0309 PINTO0309 merged commit 141c9c4 into main Apr 26, 2023
@PINTO0309 PINTO0309 deleted the fix_cind_kat branch April 26, 2023 03:43
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.

1 participant