-
Notifications
You must be signed in to change notification settings - Fork 18
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
consider making funnel.Rmd a part of large.Rmd #515
Comments
And re-organize content.
|
I'll have a go at this. |
Thanks. I welcome proposals to further split up these vignettes. Right now, both "large" and "funnel" are almost too large on their own, and it would be too scary for me to attempt to read both in one sitting. What's the motivation to merge these? |
The motivation is that both are meant for large data. I think that with better subsection titles, it'll be easier to skim. |
Actually I might split them but differently!
|
|
@krlmlr I'd consider making funnel a part of "how to compute" so there'd be three vignettes instead of two.
I think the paragraph comparing duckplyr to dbplyr that is currently in the large.Rmd vignette should be moved to the README (I can make a PR) because comparisons to similar tools are sort of usual in READMEs. I'd remove the "funnel" section from the vignette for developers. How does this sound? |
I'd need to think about it. Let's focus on the small wins for now. |
In my view, data in and data out should be in one vignette. We can add a third vignette that presents an entirely different view, perhaps a "Getting started" vignette that links to the two others. If there's then material that no longer is a good fit for "large" or "funnel" (soon to be "valve"?), we can rearrange gradually. |
Would you like to contribute a "Getting started" vignette? |
Yes, I'll try! 😸 I'll share the diagram source then, as it could go in there. |
No description provided.
The text was updated successfully, but these errors were encountered: