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

Bugfix #36

Merged
merged 1 commit into from Mar 3, 2018
Merged

Bugfix #36

merged 1 commit into from Mar 3, 2018

Conversation

@adamjohansen
Copy link
Collaborator

@adamjohansen adamjohansen commented Mar 3, 2018

A fix to address #34 and to make the simulation code and particle filter use the same default value for the initial state variance.

Nothing controversial is there (mainly aimed at @eddelbuettel and @LeahPrice)?

…eak to the initial state variance parameter in pfNonlinearBS to make it consistent with the code for simulating data from this model.
@eddelbuettel eddelbuettel merged commit 7d6bb94 into rcppsmc:master Mar 3, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@LeahPrice
Copy link
Collaborator

@LeahPrice LeahPrice commented Mar 3, 2018

This all looks good to me. Thanks Adam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.