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

Set default CMake options used by Storm #8

Merged
merged 2 commits into from
Apr 24, 2023

Conversation

volkm
Copy link
Contributor

@volkm volkm commented Apr 16, 2023

Sets the CMake options -DUSE_CLN_NUMBERS=ON -DUSE_GINAC=ON -DTHREAD_SAFE=ON as default.

@tquatmann
Copy link
Member

Since CLN and GINAC still don't appear to work on apple silicon, can we set this default conditionally?

@volkm
Copy link
Contributor Author

volkm commented Apr 17, 2023

Thanks @tquatmann and @sjunges for indicating the issue on Apple Silicon. I added a check for it.
Can somebody of you test it on Apple Silicon for me?

@tquatmann
Copy link
Member

I just tested, it works!
Both carl and storm now work with default cmake options (for me)
I approve merging.

@volkm volkm merged commit 58fc2aa into moves-rwth:master Apr 24, 2023
@volkm volkm deleted the default_settings branch April 24, 2023 12:45
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