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

Temporal forge needs max_tracelength to be set before min_tracelength #274

Open
dominicmkennedy opened this issue Aug 29, 2024 · 1 comment

Comments

@dominicmkennedy
Copy link

Temporal Forge code:

#lang forge/temporal

option min_tracelength 6
option max_tracelength 6

Error message:

Cannot set min_tracelength to 6 because min_tracelength cannot be greater than max_tracelength. Current max_tracelength is 5.
@tnelson
Copy link
Owner

tnelson commented Oct 23, 2024

I think the issue title is slightly off, since it refers to max twice, rather than min and max.

But setting that aside, this is annoying, I agree. But the problem is one of atomicity. Without the following max_tracelength statement, the first would put the system in a state that violated the invariant max_tracelength >= min_tracelength. Forge doesn't do lookahead, and there's not at present a way to make atomic changes to multiple options at once. I'm happy to take suggestions, though.

@bennn bennn changed the title Temporal forge needs max_tracelength to be set before max_tracelength Temporal forge needs max_tracelength to be set before min_tracelength Nov 5, 2024
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