-
Notifications
You must be signed in to change notification settings - Fork 10
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
archived on CRAN? #115
Comments
Would be great to get this fixed. The older packages on CRAN have the same dependency. |
This is really an "upstream" problem with the In the meantime you should be able to use either e.g. for (p in c("leeper/prediction", "leeper/margins", "fsolt/dotwhisker")) {
remotes::install_github(p)
} I don't think any of these packages have compiled code, so you shouldn't even need to have developer tools installed ... |
Thank you, that's very helpful - I had already tried remotes::install_version() but it failed due to the missing margins package. Will give these instructions a try. |
Just submit the new version to CRAN. Have a try to install from there, @LucieCBurgess . Let me know if there's any problem. Sorry for not addressing this issue sooner... |
Apparently, I missed the last email from Thomas Leeper with Ben ... My bad. And sorry, @LucieCBurgess, it looks like we need to wait for Thomas to update the upstream package If he still cannot release the new |
In private e-mail Thomas said he had been busy with personal stuff but was hoping to re-submit |
@bbolker Thanks for you advise, |
The CRAN page says:
However, I can't for the life of me see where this dependence could come from. Neither theDESCRIPTION
file of the last-archived version on CRAN, nor the version on github, seem to show any signs of importingprediction
... ?? I suppose there's a possibility of a recursive dependence, but in that case why wouldn't the direct dependence be reported?Thanks to
r-package-devel
, I now know that this is through the dependence on themargins
package. Short of (1) removing this functionality, (2) switching to the more activemarginaleffects
package, I don't see a fix other than waiting for the maintainer of theprediction
package to resubmit to CRAN ...The text was updated successfully, but these errors were encountered: