Skip to content

Fix profiling SeparableConv1D and SeparableConv2D

Javier Duarte requested to merge github/fork/qberthet/fix_890 into main

Created by: qberthet

Description

As described in #890 (closed), profiling model using SeparableConv1D or SeparableConv2D fail. The application of theses two commit fix the test cases decribed in the issue.

  • The base issue is similar to #829, which have been fixed by #833. The first commit of this PR contains very similar changes.
  • The second commit fixes a subsequent issue where a list is used where a dict is expected. I am less sure about this one, maybe it hides something more deep, or it fixes another bug not yet reported. Experienced eyes required here.

closes #890 (closed)

Type of change

  • Bug fix (non-breaking change that fixes an issue)

Tests

Tests explaining how to reproduce the problem have been provided in the original issue #890 (closed).

Checklist

  • I have read the guidelines for contributing.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have installed and run pre-commit on the files I edited or added.
  • I have added tests that prove my fix is effective or that my feature works.

Merge request reports