-
Notifications
You must be signed in to change notification settings - Fork 16
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
Allow disabling limits #106
Conversation
Pull Request Test Coverage Report for Build 10098350690Details
💛 - Coveralls |
@domrachev03 @simeon-ned FYI, this PR makes limits similar to tasks. Users will be able to disable them too if they want (by setting Since we are about to release Pink v3 I'd rather make this API update now. Feel free to share your thoughts. I also have one question: in barriers there is a |
@stephane-caron I think providing limits as you have done in this PR is a clearer way, also I would prefer to have |
Will add with acceleration limits
ac4d5fc
to
de898eb
Compare
This PR follows from this comment.