Closed IndrajeetPatil closed 2 years ago
Awesome!
Do you have plans to submit the new version of the package to CRAN anytime soon?
The docs also need to be tweaked a bit, btw:
Ah, thank you for the reminder! That will be fixed in the released version. I'll submit it to CRAN right now. Sorry for not leaving a message before or immediately after closing the issue---the entire morning was spent cleaning the repo (issues piled up).
Thank you again, Indrajee @IndrajeetPatil , for the suggestion. And I would also like to say, the broomExtra
is amazing! Although we won't use it directly in dotwhisker
as the author suggested (how nice, friendly, and open-mind the author is!), it definitely helps people to prepare their analyses outputs for using dotwhisker
. It also opens opportunities for presenting pretty tables (in the appendices of course :P) with modelsummary
!
Sounds good.
Thank you very much for being open to suggestions and for all the work you have put into this awesome visualization package! :)
Is there any downside to making this switch? None, the outputs are pretty much identical because
broomExtra
defaults toparameters
anyway. So you will get the same behavior you do now.Benefit: It reduces the dependency load significantly!
Created on 2021-06-17 by the reprex package (v2.0.0)
P.S. I am author and maintainer of
broomExtra
, and would be happy to make a PR if you want.