-
Notifications
You must be signed in to change notification settings - Fork 28
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
Third attempt to merge #191 #307
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #307 +/- ##
==========================================
- Coverage 86.25% 86.17% -0.09%
==========================================
Files 73 73
Lines 16537 16677 +140
==========================================
+ Hits 14264 14371 +107
- Misses 2273 2306 +33
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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 if tests will pass
pytest took 514.14s on this branch and 512.94s on master, so it seems the checks are good |
All changes to
src/exo/LoopIR_scheduling.py
except for new primitives.All changes to
tests/test_schedules.py
.All changes to
src/exo/LoopIR_unification.py