Andrew Johnson
Andrew Johnson
> The fix is that somewhere we need to convert falsy values at the R level to completely unset the ENV variable at the bash/cpp level No we shouldn't do...
Apologies for the extended delay @katrinabrock! I'm catching back up on Stan dev this week, and have better access to an OpenCL system now. I'll put aside some time to...
The approach I took for `cmdstanr` for compatibility with both was to [convert any `true`/`false` to 0/1](https://github.com/stan-dev/cmdstanr/blob/0e3a99a52bc5ad15d9e386e9993c10ad6905818b/R/csv.R#L843), so that the parsing logic for the rest stayed the same
> Note that using cmdstanr with brms relies on read_stan_csv. [It doesn't look like that's the case anymore](https://github.com/paul-buerkner/brms/blob/2917e72ec0cbd9cd8ade6465cf44cfaf7c457461/R/backends.R#L704) (which makes sense, otherwise there would have been a ton of bug...
Great to hear that the package could be useful! I do have plans to expand the package's functionality, with the end goal of being a drop-in alternative for V8 (where...
Also as an aside, you could avoid bundling the `quickjs` sources/engine by just adding `LinkingTo: QuickJSR` in your package description. This will save you some headaches with CRAN, since the...
@wch Thanks for all of these suggestions, and I agree! I'll be starting on updates this week, so will be working through these. > What do you think about changing...
I don't think we'd be able to treat an array of `0/1` ints as booleans for indexing here unfortunately, since it would be ambiguous whether an array of `1` should...
There's some previous discussion around a `bool` type here: https://github.com/stan-dev/stanc3/issues/1328, essentially treating the type as syntatic sugar for `int` for backwards compatibility
@georgestagg apologies for the ping, but would it be possible to have a look at this PR? I'd like to fix building/linking for packages using `RcppParallel` and the TBB