-
Notifications
You must be signed in to change notification settings - Fork 131
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
Feature request: Add support for add_ci()
with tbl_summary(percent = 'row')
#1929
Comments
I started implementing this in This would be best implemented in |
Any timeframe on this? This used to work (add_ci with percent=cell) in an older version of tbl_summary (1.7.2) but is no longer. I have several workflows that used this to generate row-wise confidence intervals and have not been able to get a backwards compatible installation of R/all my libraries to use the older version. Thank you - I love this package and use it extensively. |
Hi @elizn9 , I didn't realize that functionality was available in the earlier version. 😬 I'll get to this when i can, but I honestly don't know when I'll have the bandwidth. In the meantime, I recommend using {renv} to ensure your projects use the needed versions of packages. You can also use tools like Posit Package Manager which has dated archive of all CRAN packages. You can easily install older versions as needed https://packagemanager.posit.co/client/#/repos/cran/setup?snapshot=2024-02-01 |
Hi @ddsjoberg - Thanks for the response. Unfortunately package management is not possible when working on externally managed servers that host R (and update libraries on an adhoc basis). But, I figured out a workaround using tbl_custom_summary to build proportion tables and then merging them all together. It's less elegant, but gets what I need. But when/if you get to it, it would be much appreciated. It definitely worked in the previous (pre 2.0) version. Thanks! |
add_ci()
with tbl_summary(percent = c('row', 'cell'))
add_ci()
with tbl_summary(percent = 'row')
@elizn9 I have a pull request prepared to implement this functionality. Can you please install this version and give it a try before it's added to the package? You can install it with: |
Thank you - Yes! It seems to work correctly for row % CIs (local installation of R). I have not tested cell CIs as I do not generally use these. I will try to get this version installed on my remote server (R) and test with other project code (this week).
On 2025-01-05 7:17:17 PM, Daniel Sjoberg ***@***.***> wrote:
@elizn9 [https://github.com/elizn9] I have a pull request prepared to implement this functionality. Can you please install this version and give it a try? You can install it with: ***@***.***_ci-denom", dependencies = TRUE)
—
Reply to this email directly, view it on GitHub [#1929 (comment)], or unsubscribe [https://github.com/notifications/unsubscribe-auth/A74E6SWWNUJOI6ONJFS5POL2JHYT3AVCNFSM6AAAAABNESRDDSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZSGIYDCMBYGQ].
You are receiving this because you were mentioned.Message ID: ***@***.***>
[d7ed3354-e578-470c-b8b8-74fc6a5079d5]
|
We will also need to support
add_ci.tbl_svysummary()
The text was updated successfully, but these errors were encountered: