Brian Ward

Results 739 comments of Brian Ward

For testing I ended up using the same blocker1.csv and blocker2.csv as the stan tests, but for initially getting them to agree I was just using an arbitrary CSV file,...

I suspect it is because this is the only game he was on the active roster, he was optioned back to their Triple-A team the next day. If you look...

Which is the worse practice here, the above apparmor exception or disabling sandboxing?

In the past we've talked about re-using `` for this, so your example would instead look like ```stan model { cholesky_factor_corr[N] L; // use default no annotation necessary cholesky_factor_corr[N] L2;...

As an additional note: besides just the new type, we will also want to document the new overloads to functions like `sum_to_zero_constrain` on [their page](https://github.com/stan-dev/docs/blob/master/src/functions-reference/transform_functions.qmd)

It's all merged now -- @spinkney do you want to take the first crack? I can add the new function overloads to whatever branch you work on

While we're at it, there's a typo in the sum to zero vector: https://discourse.mc-stan.org/t/constraint-transforms/39384

Here's what that could look like on Stan's homepage (just the minimal example, no demo (yet)) ![Image](https://github.com/user-attachments/assets/db327a7b-f653-4ad8-99c6-420683e34924) ![dark mode](https://github.com/user-attachments/assets/a5864d4c-3faf-4e58-8499-7882011efcd8)

Thanks @avehtari. Writing that test made me realize something else: The `y_index` argument to `laplace_marginal_poisson_log_lpmf` is currently 0-indexed rather than 1-indexed. Is that intentional @charlesm93 @SteveBronder?

@charlesm93 y_index is now 1-indexed and I removed the newly redundant poisson_2_log helper