-
-
Notifications
You must be signed in to change notification settings - Fork 3
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: Compatibility with databases via dbplyr
#378
Comments
Thanks for the post @kenkomodo ! That is an interesting point you bring up. In dbplyr, all the calculations occur in the data base, correct? I am not sure we'd be able to generalize for that to work well? Our simple tabulations are processed through BUT, this is not a topic I've spent much time researching! Perhaps you can correct me if my assumptions are incorrect? |
@ddsjoberg Yes - Would it still require a large refactor since the database As for the functions that don't have equivalents in SQL, I think that would simply be a limitation and the an error message could be printed in the console for the user if they tried to use a function that didn't have an equivalent in SQL. |
Ahh very interesting! I can certainly see how that would be useful for some key tabulations. For now, I think that is going to fall out of the scope of cards. But if you choose to spin up a package to create DB ARDs, we're certainly here to help in the design and to explain our design choices (would be great to keep them consistent)! |
What's the feature?
It would be really great to have compatibility with
dbplyr
summary functions so that users could easily create ARDs or generate tables viagtsummary
via thecards
dependency in cases where the data to be summarized are larger than memory on either their local computer or in a cloud-based RDMS.Since
dbplyr
has a lot of analogous verbs compared todplyr
and a good portion of functions can be translated bydbplyr
, my guess is it wouldn't be a giant lift (although I could easily be wrong). I just haven't had time to get familiar enough with thecards
source code to know exactly which pieces need changed.The text was updated successfully, but these errors were encountered: