Jamie Gilbert

@mvankessel-EMC thanks for this - I'm not quite sure what we use the json for? @gowthamrao is it used in the meta data to check what the calling arguments set...

Yes - i looks like this is just stored in the metadata result - we just need to create a list that stores the relevant arguments (and doesn't leak user...

The calculation has been fixed as per screenshot below: however, I'm keeping this issue open as currently the operation is slow as legacy code pulls down data the target and...

This is highly desirable but a large amount of work and suitable to pick up for a few developers interested in learning this package. Ideally, we should test the functions...

Goal - have all unit tests to run in under 10 minutes

The unit tests have been greatly sped up on database platforms by sampling cohorts, however, this will require a release of CohortGenerator first

@mdlavallee92 Is this using `devtools::test`? `devtools::load_all` will break the SqlRender loader for some reason related to how it finds the files. An easy workaround on linux/mac is to create the...

@mdlavallee92 - To the main point of this question. I do intend to expose the functions for use outside of the package, however I can't give a clear timeline. I...

@mdlavallee92 I completley agree with the principle of separating this out into distinct parts of a pipeline. Part of the reasoning behind RMM was to allow people to begin extending...

So this is a bit of an annoying one. The tidyselect package started throwing warnings for using `.data$` in dplyr calls, removing these now creates this problem. It is possible...