-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
report initialization as first draw #547
Comments
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: used by CmdStan here: cmdstan/src/cmdstan/command.hpp Line 189 in a22802c
ignored by services methods here: https://github.com/stan-dev/stan/blob/6d35a762bf56c72fef574302e5af4b9c70703625/src/stan/services/util/initialize.hpp#L209 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! |
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
The text was updated successfully, but these errors were encountered: