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

add LFO vignette #90

Merged
merged 2 commits into from
Oct 16, 2018
Merged

add LFO vignette #90

merged 2 commits into from
Oct 16, 2018

Conversation

paul-buerkner
Copy link
Contributor

This PR adds a vignette about leave-future-out cross-validation based on our case study.

@avehtari
Copy link
Collaborator

Is this the same as in html in m-step repo? No need to knit this version?

@paul-buerkner
Copy link
Contributor Author

I also have the knitted version ready on a branch, but it seems as if the loo master branch contains no knitted version of any vignette. That's why I didn't include it. Would love to have @jgabry in this discussion to tell us about where/when to add the knitted version.

@paul-buerkner
Copy link
Contributor Author

After some discussions with Aki and some further changes, this vignette is ready to be included.
I will later move it to gh-pages in order to update the website accordingly.

@paul-buerkner paul-buerkner merged commit e358940 into master Oct 16, 2018
@jgabry
Copy link
Member

jgabry commented Oct 18, 2018

Thanks Paul!

@jgabry jgabry deleted the lfo-vignette branch October 18, 2018 16:01
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.

3 participants