-
-
Notifications
You must be signed in to change notification settings - Fork 369
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
Fix and add unit tests for issue 3071 - adaptive sampler runs on models w/ zero params. #3303
Conversation
Jenkins Console Log Machine informationNo LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.3 LTS Release: 20.04 Codename: focalCPU: G++: Clang: |
This is causing a failure on jenkins in another pr in the math library. Can we revert this, update it to develop, and open another pr? |
@SteveBronder that failure was from before this was merged, does it still happen now? |
Submission Checklist
./runTests.py src/test/unit
make cpplint
Summary
PR #3071 had a failing unit test; this PR fixes that test and adds a test to demonstrate that the adaptive sampler can be run on a model with zero parameters.
Intended Effect
Interfaces don't need to enforce calls to "fixed_param" sampler for models with zero params.
How to Verify
Unit tests
Side Effects
N/A
Documentation
N/A
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): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses: