-
Notifications
You must be signed in to change notification settings - Fork 176
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
refactor: Release step constraint after stepper in propagation #2596
refactor: Release step constraint after stepper in propagation #2596
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #2596 +/- ##
==========================================
- Coverage 49.56% 49.54% -0.02%
==========================================
Files 474 474
Lines 26930 26929 -1
Branches 12423 12425 +2
==========================================
- Hits 13348 13343 -5
- Misses 4748 4749 +1
- Partials 8834 8837 +3 ☔ View full report in Codecov by Sentry. |
…pagator-release-step-contrained
…diwand/acts into refactor-propagator-release-step-contrained
…pagator-release-step-contrained
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.
Looks reasonable. There's still some CI failures at this point.
did't update the reference files yet to give people a chance to look at the physmon diff. will update now |
…pagator-release-step-contrained
This avoids cases where an actor/aborter sets a step constrained that will never be overwritten again.
I also generalize the
release
function and rename theset
function in the stepper.blocked by