Skip to content

Clarify meaning of "set flag" in cmdstan-guide#808

Merged
WardBrian merged 2 commits intostan-dev:masterfrom
katrinabrock:fix-cmdstan-1293
Aug 21, 2024
Merged

Clarify meaning of "set flag" in cmdstan-guide#808
WardBrian merged 2 commits intostan-dev:masterfrom
katrinabrock:fix-cmdstan-1293

Conversation

@katrinabrock
Copy link
Contributor

Submission Checklist

  • Builds locally
  • [N/A] New functions marked with <<{ since VERSION }>>
  • Declare copyright holder and open-source license: see below

Summary

State explicitly that setting STAN_THREADS, STAN_MPI, and STAN_OPENCL to any value, including falsy values, turns the corresponding feature on.

Fix stan-dev/cmdstan#1293

Copyright and Licensing

Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Max Planck Institute of Animal Behavior

By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

State explicitly that setting `STAN_THREADS`, `STAN_MPI`,
and `STAN_OPENCL` to any value, including falsy values,
turns the corresponding feature on.

Fix stan-dev/cmdstan#1293
@WardBrian
Copy link
Member

I think "defined" is slightly more correct than "set", but this is already an improvement so I would be happy to merge it

@katrinabrock
Copy link
Contributor Author

@WardBrian done

Copy link
Member

@WardBrian WardBrian left a comment

Choose a reason for hiding this comment

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

Thanks!

@WardBrian WardBrian merged commit 9a6b9fb into stan-dev:master Aug 21, 2024
@katrinabrock katrinabrock deleted the fix-cmdstan-1293 branch August 22, 2024 11:08
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.

Doc Fix? Setting STAN_THREADS=false (or FALSE, 0, etc) results in STAN_THREADS=true

2 participants