optimizer icon indicating copy to clipboard operation
optimizer copied to clipboard
verified publisher icon onnx

Add sign to idempotent ops list

Open ananyamukh6 opened this issue 2 years ago • 2 comments

Sign op is idempotent, so adding it to eliminate_consecutive_idempotent_ops pass

ananyamukh6 avatar Aug 18 '23 05:08 ananyamukh6

LGTM, thanks!

daquexian avatar Aug 30 '23 04:08 daquexian

@daquexian , the tests seemsed to have failed because of flake8. I pused a patch fixing the flake8 errors. Is there any other style checks to be run (like say for the cpp code)?

ananyamukh6 avatar Aug 30 '23 04:08 ananyamukh6