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 #407

Merged
merged 44 commits into from
May 3, 2023
Merged

Update vignettes #407

merged 44 commits into from
May 3, 2023

Conversation

fweber144
Copy link
Collaborator

This was mainly intended to introduce the new functions ranking(), props(), and plot.props() in the main vignette, but in the end, many more changes were performed in the two vignettes (see the commit messages for details).

…sible

in the rendered document (for easier diffing).
included in the CV (but still have a comparably low runtime).

Also increase `iter` from 500 to 1000 to avoid ESS warnings and
also perform some minor improvements.
This is done to show some messages on purpose. Warnings shouldn't be suppressed
by default to avoid that problems are not noticed.
…tely valid

because of the lack of validation for the selection of a size.
`validate_search = FALSE`.

Also perform some larger improvements concerning the content.
the additional runtime is not that high, but when re-running several times, the
additional runtime becomes noticeable).
focus is accuracy) instead of `modsize_decided <- 6` (when focus is sparsity).
@fweber144 fweber144 merged commit 525e3cd into stan-dev:master May 3, 2023
@fweber144 fweber144 deleted the vignette_updates branch May 3, 2023 18:24
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

1 participant