-
Notifications
You must be signed in to change notification settings - Fork 37
[Merged by Bors] - Add requires_threadsafe
#353
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
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…threadsafe varinfo or not
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
|
Does this seem good? Up for chosing a different name too. |
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.
Maybe something more explicit like use_threadsafe_eval or use_evaluate_threadsafe? Otherwise looks good to me.
|
I like it! Done 👍 |
…namicPPL.jl into tor/threadsafe-improvement
|
bors r+ |
See related discussion: TuringLang/Turing.jl#1726 (comment).
|
Build failed: |
|
It's back! |
|
bors r+ |
See related discussion: TuringLang/Turing.jl#1726 (comment).
|
Pull request successfully merged into master. Build succeeded: |
requires_threadsaferequires_threadsafe
See related discussion: TuringLang/Turing.jl#1726 (comment).