-
Notifications
You must be signed in to change notification settings - Fork 182
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
use OpenMP for simplifying thread setup in fire particle animations (master branch) #1561
Conversation
f0d19b2
to
7021a4b
Compare
LGTM, can you rebase on top of master to get the CI working, too? |
7021a4b
to
1f87f43
Compare
updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: the CI still doesn't pass.
Ping |
1f87f43
to
4f10882
Compare
Fixed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
I will merge after the 0.8.1 release to avoid adding new dependencies for a minor release.
Signed-off-by: Aisha Tammy <[email protected]>
Signed-off-by: Aisha Tammy <[email protected]>
Notice: Wayfire's development is temporarily happening in the
stabilize-api
branch. If you open a pull request for the master branch, it will likely not be merged before the stabilize-api branch is itself merged into master.