-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
register pass compatibility #27357
register pass compatibility #27357
Conversation
Thanks for your contribution! |
5531277
to
3951be4
Compare
def test_check_output(self): | ||
self.check_output() | ||
|
||
PassVersionChecker.IsCompatible('conv_affine_channel_fuse_pass') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
需要添加断言
self.assertTrue
def test_check_output(self): | ||
self.check_output() | ||
|
||
PassVersionChecker.IsCompatible('conv_affine_channel_fuse_pass') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上
def test_check_output(self): | ||
self.check_output() | ||
|
||
PassVersionChecker.IsCompatible('conv_affine_channel_fuse_pass') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上,以下相同情况请统一修复
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
Others
PR changes
Others
Describe
conv_bn_fuse_pass
,conv_affine_channel_fuse_pass
,repeated_fc_relu_fuse_pass
,shuffle_channel_detect_pass
repeated_fc_relu_fuse_pass
when input dims is not 2