Bugfix/poisson glm - #1510
Conversation
5665c9d to
fa93a21
Compare
…stable/2017-11-14)
|
(stat_comp_benchmarks/benchmarks/gp_pois_regr/gp_pois_regr.stan, 0.99) |
|
In |
|
(stat_comp_benchmarks/benchmarks/gp_pois_regr/gp_pois_regr.stan, 0.99) |
|
(stat_comp_benchmarks/benchmarks/gp_pois_regr/gp_pois_regr.stan, 1.0) |
|
(stat_comp_benchmarks/benchmarks/gp_pois_regr/gp_pois_regr.stan, 0.98) |
bbbales2
left a comment
There was a problem hiding this comment.
Code looks good, tests looks good, merge whenever.
Summary
Fixes the include_summand bug in
poisson_log_glm.Tests
Added a test that fails on develop but passes on this PR.
Side Effects
/
Checklist
Math issue: Fixes poisson_log_glm sampling statement fails to fit #1498
Copyright holder: Rok Češnovar, the fix was suggested by Ben Bales
the basic tests are passing
the code is written in idiomatic C++ and changes are documented in the doxygen
the new changes are tested