Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Option --cut-off and --sort-by=s don't work as one may expect (documented as is) #33

Closed
Jellix opened this issue Jun 17, 2020 · 1 comment
Assignees
Labels
wontfix This will not be worked on
Milestone

Comments

@Jellix
Copy link
Member

Jellix commented Jun 17, 2020

Describe the bug
As described in the README.md, the --cut-off option works for maximal proof time, not for maximum time for successful proof.

Expected Behavior
If one requests sorting by maximum time for successful proof, the --cut-off should take that parameter into account.

@Jellix Jellix added the bug Something isn't working label Jun 17, 2020
@Jellix Jellix added this to the V1.0.0 milestone Jun 17, 2020
@Jellix Jellix self-assigned this Jun 17, 2020
@Jellix Jellix pinned this issue Jun 18, 2020
@Jellix
Copy link
Member Author

Jellix commented Jun 18, 2020

After some testing, I leave it as is. It doesn't seem to make sense to apply the cut off for successful proof attempts.

Especially if another prover failed and needs a significant higher time, we would omit that VC from the output, even though it is still way beyond the cut-off time.

@Jellix Jellix closed this as completed Jun 18, 2020
@Jellix Jellix added wontfix This will not be worked on and removed bug Something isn't working labels Jun 18, 2020
@Jellix Jellix unpinned this issue Jun 18, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

1 participant