Skip to content
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

Add gt and gtsummary examples to regression output vignette #287

Open
lukesonnet opened this issue Feb 25, 2019 · 7 comments
Open

Add gt and gtsummary examples to regression output vignette #287

lukesonnet opened this issue Feb 25, 2019 · 7 comments

Comments

@lukesonnet
Copy link
Contributor

This will be done once those packages are on CRAN (tagging @vincentarelbundock)

@lukesonnet
Copy link
Contributor Author

lukesonnet commented May 16, 2019

Is the gtsummary on cran the same package? I can't see author overlap... @vincentarelbundock

It appears not...

@vincentarelbundock
Copy link
Contributor

vincentarelbundock commented May 16, 2019

Indeed, there is an unfortunate name conflict. I released my package on github in Jan 2019, but had held off on releasing to CRAN because {gt} isn't there yet, and CRAN doesn't like off-site dependencies.

As far as I can tell @ddsjoberg developed a package called {clintable} based on {kableExtra} tables. A couple weeks ago, he appears to have changed the name of his package to {gtsummary}. As far as I can tell (I may be wrong), that package doesn't actually use {gt} yet, so perhaps they released in anticipation of using {gt} in the future (name squat?).

The name conflict has already led someone to raise an issue on my repo; which...argh! Given that I released my package as "gtsummary" publicly before, my preferred resolution would be for @ddsjoberg to change the name back to the original (or to pick another one), but I obviously don't have any control over that. I sent him an email 4 days ago and am waiting for a reply.

@lukesonnet

@acoppock
Copy link
Collaborator

Ouf, that's a really unfortunate situation and I'm really sorry to hear about this conflict. @lukesonnet let's hold off until this is resolved. Thank you @vincentarelbundock

@lukesonnet
Copy link
Contributor Author

Any update here? Sorry for the conflict, I just am curious about how best to support this going forward.

@vincentarelbundock
Copy link
Contributor

Thanks for following up!

The author does not sound receptive beyond "hoping the conflict does not cause confusion in users" (it already has, obviously). It looks like I'll have to change the name of my package.

I was thinking about modelsummary with a shortcut to ms.

What do you think? Any other suggestions?

@vincentarelbundock
Copy link
Contributor

@lukesonnet @acoppock

Sorry this took so long, but I just wanted to let you know that the package is now available from CRAN as {modelsummary}.

Thanks again for your interest. Please let me know if you have any feature requests or if you find any bugs.

@lukesonnet
Copy link
Contributor Author

Great! Thanks! We're still gonna wait to push this out until gt is on CRAN.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants