Removed soft-deprecated code from posterior_vs_prior. - #658
Open
tjmckinley wants to merge 5 commits into
Open
tjmckinley wants to merge 5 commits into
tjmckinley wants to merge 5 commits into
Conversation
Member
Thanks for the PR and for this comment. That was our goal when we developed the package! Looking at your PR made me realize we have other deprecated ggplot2 syntax that we need to clean up so I'm going to push a few commits to this before merging. @bgoodri while fixing deprecated ggplot2 syntax I also fixed a bug in |
Plotting a bivariate smooth from a model with more than one smooth term either errored or silently used the wrong variables: - `sapply()` returned a list or matrix for `xnames` (depending on the mix of smooth dimensions) and only the vector case was handled. - `fs` and `xnames` were subset by a logical the length of `smooths` rather than `labels`, so the first smooth's variables were always used regardless of which one was selected. - `nd` was resampled with indices up to `nrow(xz)` (10000) rather than `nrow(nd)`, producing NA rows and a NaN surface whenever the model had other covariates. - The bivariate check was model-wide (`!any(fs)`), so any `bs = "fs"` smooth sent bivariate smooths down the univariate path.
Author
|
Awesome. Glad this was useful. Thanks again for the package. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
Thanks for developing
rstanarm, it's made teaching Bayesian methods far easier and more accessible! I noticed a warning when running theposterior_vs_prior()function for more recent updates oftidyversepackages, because it relies on some soft-deprecated functions. A reprex is below:This returns:
This PR contains a possible fix utilising
rlang, which is a dependency ofggplot2, which itself is a dependency ofrstanarm, so hopefully this should work without any further changes. Not sure if this is useful, but the code is here if it is. It seems to fix the problems for me.Cheers,
TJ