Skip to content

add LFO vignette#90

Merged
paul-buerkner merged 2 commits into
masterfrom
lfo-vignette
Oct 16, 2018
Merged

add LFO vignette#90
paul-buerkner merged 2 commits into
masterfrom
lfo-vignette

Conversation

@paul-buerkner

Copy link
Copy Markdown
Contributor

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

@avehtari

Copy link
Copy Markdown
Member

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

@paul-buerkner

Copy link
Copy Markdown
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
Copy Markdown
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

jgabry commented Oct 18, 2018

Copy link
Copy Markdown
Member

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