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

Update Vignettes #254

Merged
merged 10 commits into from
Nov 8, 2023
Merged

Conversation

TahminaMojumder
Copy link
Contributor

No description provided.

@TahminaMojumder
Copy link
Contributor Author

should there be an example for convergence check??

gelmanDiagnostics(out, plot = T)

There are several places where eval = F - is this intentional?

## Definition of likelihood function

@florianhartig
Copy link
Owner

Hi Tahmina,

The Gelman Diagnostics are the convergence checks - if you want, you can clarify this in the text.

The eval = F is correct, the code is not supposed to run - if you think that clarifies things for the reader, you could add a comment that this is not excecutable R code, but just pseudocode that shows the principle in which one should program the functions.

@TahminaMojumder
Copy link
Contributor Author

Write here some links that were useful to me to understand MCMC

@florianhartig
Copy link
Owner

Hi @TahminaMojumder , I'm looking at merging this branch, but can you still revert all line breaks that you added? I think this is easier for the future!

@florianhartig
Copy link
Owner

Hi @TahminaMojumder - I think you said you changed back the line breaks, but I still see them - did you commit this?

@TahminaMojumder
Copy link
Contributor Author

For Rmd files, when I changed from source to visual to source again. Then the changes in line break I made changed again to previous version. That is how the changes I made was not committed. Now I have fixed this.

@florianhartig florianhartig merged commit eef2476 into florianhartig:master Nov 8, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants