Open bob-carpenter opened 7 years ago
Thanks for tagging this as v3.0.0. I don't think we can change existing behavior because it'll be a off-by-one nightmare everywhere, but if we do split to multiple files, this will be easy.
I just tagged @betanalpha just to see if he has any thoughts.
Thanks Bob and Daniel. I'll create a similar test case for Stan.jl to track this issue and the other outstanding 3.0.0 issue (save-warmups and thinning, #510 ).
Adding this initial value to the saved warmups samples I guess is just for educational purposes, as Chris was using it?
there's an unused argument to all service call methods: init_writer
which seems to be intended to capture the initial parameters on the unconstrained scale.
used by CmdStan here: https://github.com/stan-dev/cmdstan/blob/a22802cc10fce329036b164ee9c6bd4cbaa7aebc/src/cmdstan/command.hpp#L189
ignored by services methods here: https://github.com/stan-dev/stan/blob/6d35a762bf56c72fef574302e5af4b9c70703625/src/stan/services/util/initialize.hpp#L209 (but this is after the unconstraining xform - we'd want to output the first draw on the constrained scale).
at this point, we've got enough hooks on the model class that we could hook this up to its own file and output this info.
This would very much help diagnostics, thank you! (See https://discourse.mc-stan.org/t/does-the-init-argument-work-with-cmdstanr/17255/11 for example, or the hour I just spent trying to understand why I didn't find my init values in the warmup 😅)
Summary:
The first draw in the output summary if warmup is saved should be the initialization.
Description:
It currently doesn't.
Reproducible Steps:
Program:
Current Output:
Output of
is
Expected Output:
First value of y should be 0.
Additional Information:
Provide any additional information here.
Current Version:
v2.15.0