Skip to content
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

Failed sampling runs should not be cached #452

Closed
riddell-stan opened this issue Sep 10, 2020 · 1 comment · Fixed by #516
Closed

Failed sampling runs should not be cached #452

riddell-stan opened this issue Sep 10, 2020 · 1 comment · Fixed by #516

Comments

@riddell-stan
Copy link
Contributor

The test test_sampling_initialization_failed illustrates the problem. Or at least I think it does. It fails regularly (but not always) on the second run.

@riddell-stan riddell-stan added this to the 3.x (pystan 3 beta 4) milestone Oct 5, 2020
@riddell-stan riddell-stan pinned this issue Nov 30, 2020
@stale
Copy link

stale bot commented Dec 4, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Dec 4, 2020
@stale stale bot closed this as completed Dec 12, 2020
@riddell-stan riddell-stan reopened this Dec 12, 2020
@stale stale bot removed the wontfix label Dec 12, 2020
riddell-stan added a commit to riddell-stan/httpstan that referenced this issue Dec 13, 2020
Do not save any messages from failed sampling runs.
For example, if sampling starts but cannot continue because attempts
to find appropriate initial values are rejected, there will be some
(logger) messages saved to disk. The presence of these messages
confuses the cache machinery, which assumes the presence of messages
on disk indicates a successful fit. Deleting them avoids the problem.

There remains the problem of retrieving the logger messages associated
with the failed fit. It's possible that this is something someone might
want to do. Future work might address this. Such a user is also free to
use CmdStan.

Closes stan-dev#452
@riddell-stan riddell-stan unpinned this issue Dec 13, 2020
riddell-stan added a commit that referenced this issue Dec 14, 2020
Do not save any messages from failed sampling runs.
For example, if sampling starts but cannot continue because attempts
to find appropriate initial values are rejected, there will be some
(logger) messages saved to disk. The presence of these messages
confuses the cache machinery, which assumes the presence of messages
on disk indicates a successful fit. Deleting them avoids the problem.

There remains the problem of retrieving the logger messages associated
with the failed fit. It's possible that this is something someone might
want to do. Future work might address this. Such a user is also free to
use CmdStan.

Closes #452
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant