-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Export zoo
method for model_parameters
generic
#804
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jimrothstein Thanks!
This looks good, overall. Can you rebase with the main branch and see why there are changes to .Rd
files? That is not expected.
Feel free to make these changes here, or open a new PR.
Got the latest version (git pull upstream main) and then added TWO LINES: model_parameters.zoo <- model_parameters.default` Look any better? |
Yes, but there is still the issue of changes to the |
On my local machine, I get 28 warnings when I document(). Need to install all the easystats packages, I think. And Sorry, not as easy as I first thought. |
Sorry for the troubles. Try running: install.packages("easystats")
easystats::install_latest() Refresh your R session and then try again to re-document. |
Doing it now ... 16 warnings .... looks like need to manually install more packages. more dependencies ... could take some time. I am US west coast time (Oregon) and getting late. Will finish later. also needs some Ubuntu packages ... I'll get there. |
Have all packages .... document() .... no warnings. Also ?model_parameters.zoo has no help. Not sure about *.Rd files More tomorrow. |
zoo
method for model_parameters
generic
Looks good now. I will merge this once the builds pass. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @jimrothstein! :)
Feel free to work more extensively on #746 in a new PR. You know the deal now. ;)
Will do ! Later today. |
Part of #746