Skip to content
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

Add default value to PrioritizedBSyncEventSelectionStrategy #115

Closed
achiyae opened this issue Sep 25, 2020 · 0 comments
Closed

Add default value to PrioritizedBSyncEventSelectionStrategy #115

achiyae opened this issue Sep 25, 2020 · 0 comments

Comments

@achiyae
Copy link
Member

achiyae commented Sep 25, 2020

The default priority value of a bsync is int.min however it should be 0 and also allow the user to set it.

The reason is that in verification, for example, I wish to simulate external event as event with the lowest priority but I cannot do that since all other events already have the lowest value.

Moreover, in many cases you think on priorities as a minimization problem and not maximization, 0 lets you do both...

This issue is probably valid for PrioritizedBThreadEventSelectionStrategy

@michbarsinai michbarsinai added this to the Pre-1.0 cleanup milestone Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants