-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Prepare CRAN release #369
Prepare CRAN release #369
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Ok, the final fine-tuning is done, I think you can submit... |
This comment was marked as outdated.
This comment was marked as outdated.
will submit today :) |
Great! There's a minimal time pressure for me, as packages ggeffects (and as a consequence, also insight) are broken due to the marginaleffects update, and I need to submit updates to CRAN, too. And ggeffects now relies on modelbased for contrasts, because I don't want to do all the work we did here for modelbased twice when there are substantial changes in marginaleffects 😬 |
it has been submitted 40min ago, fingers crossed it passes the checks 😄 |
Thanks, on its way to CRAN. Team, this breaks 'report'. |
@rempsyc 😬 |
by
argument #373marginalize
and its options #374by
matters? vincentarelbundock/marginaleffects#1373 gets fixed for CRAN submission 😎 (and add tests when fixed)