Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

re-using table names requires frequent re-running of tagme() #36

Open
plorch opened this issue Dec 9, 2020 · 0 comments
Open

re-using table names requires frequent re-running of tagme() #36

plorch opened this issue Dec 9, 2020 · 0 comments

Comments

@plorch
Copy link

plorch commented Dec 9, 2020

Next time a major revision of the MotusRBook is done, it would be great to see at least some cases where the same table name or data.frame is used for a filtered or mutated table removed. I think for most users with big projects this creates problems. Most of us are probably following the examples in the book pretty closely to construct our workflows, especially as major changes happen to motus package code. With this kind of table name re-use, it is easy to end up with very hard to track problems. The alternative (that you have used in several later chapters) is to not re-use the tables and data.frames and to just run code that goes out to the database again. For big projects, this can require a very long wait for a result. It would seem better to use different table names and then, in later examples, use those tables that are already in memory. I realize that some are jumping into the book in the middle, but I think that problem can be solved with a backward reference to the section where the selected/filtered/mutated table was created.

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

No branches or pull requests

1 participant