Skip to content

add real-time output for TLC script + add posibility to configure wor… #1650

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

Merged
merged 2 commits into from
May 5, 2025

Conversation

see-quick
Copy link
Contributor

…kers count

This PR updates the TLC script to also print real-time output + adds the possibility to configure the number of workers.

Copy link
Collaborator

@bugarela bugarela left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks for opening the PR!

@@ -10,6 +10,7 @@ source "$SCRIPT_DIR/common.sh" || { echo "Error: Could not load common.sh"; exit
APALACHE_JAR="$HOME/.quint/apalache-dist-0.47.2/apalache/lib/apalache.jar"
JAVA_OPTS="-Xmx8G -Xss515m"
FILE="" # Mandatory: User must specify a file
WORKERS=12 # Default worker count
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally, this should be an optional parameter, as TLC's default is to find the number of workers for you according to available cores. I'm fine with keeping this for the script though, we can improve that once we have this in Quint itself.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree. :)

@bugarela bugarela merged commit d480242 into informalsystems:main May 5, 2025
22 checks passed
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

Successfully merging this pull request may close these issues.

2 participants