-
Notifications
You must be signed in to change notification settings - Fork 116
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
#17863: Add sweep test for Batch Norm #18688
Conversation
82a0b62
to
d948603
Compare
adfebf2
to
dbb139b
Compare
387f1ba
to
8c9653a
Compare
dbb139b
to
c4d5a9c
Compare
c775e4d
to
b17a5aa
Compare
c4d5a9c
to
dc3aed0
Compare
b559ab5
to
81dbc2d
Compare
} | ||
|
||
|
||
def run( |
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.
Please add a test_batch_norm method to test locally via pytest.
See tests/sweep_framework/sweeps/matmul/short/matmul_traces.py
for an example.
81dbc2d
to
34d1d31
Compare
1570c9e
to
57ed0ca
Compare
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.
Could you run the sweep on pipelines?
57ed0ca
to
33f3e88
Compare
Ticket
#17863
Problem description
Add sweep test for BN
What's changed
Checklist