glow icon indicating copy to clipboard operation
glow copied to clipboard

[GraphOptz] Merge Pad into CWQConv and Pool.

Open mciprian13 opened this issue 4 years ago • 10 comments

Summary

  • Merge Pad into ChannelwiseQuantizedConvolution: put logic into the existing graph optimization pass MergePadIntoConvolution which was merging the Pad only into the Convolution node.
  • Merge Pad also into MaxPool and AvgPool by adding another MergePadIntoPooling optimzation pass.

Fixes Fixes #5706.

Test Plan Unit test in GraphOptzTest.

mciprian13 avatar Jun 08 '21 18:06 mciprian13

@jfix71 Can you review this?

mciprian13 avatar Jun 09 '21 15:06 mciprian13

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 13 '21 01:07 stale[bot]

ping

mciprian13 avatar Jul 13 '21 10:07 mciprian13

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Aug 21 '21 04:08 stale[bot]

Ping.

mciprian13 avatar Aug 23 '21 09:08 mciprian13

@jfix71 Any other comments on this PR?

mciprian13 avatar Aug 24 '21 17:08 mciprian13

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Mar 02 '22 10:03 stale[bot]

Ping.

mciprian13 avatar Mar 02 '22 13:03 mciprian13

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Apr 16 '22 14:04 stale[bot]

ping

mciprian13 avatar Apr 18 '22 09:04 mciprian13