Ben Straub
Ben Straub
> @bms63 any user could hardcode the subject identifiers instead if they were only ever going to use `STUDYID`, `USBUJID` here, but for our package to follow our own instructions...
and thanks @ProfessorP-beep for seeing this through!! This is how it works on the team :) We do PRs and I get a little huffy and puffy and ask funny...
@bundfussr @rossfarrugia @manciniedoardo - This update isn't critical to me for 1.1. Can we implement for 1.2 as it is actually quite large and seems to be opening lots of...
Hi @ProfessorP-beep, we can talk about this tomorrow, but going to pause on this PR until after we ship 1.1 release. This will be our first PR into 1.2!
Can you add them on in the data package? This is a request from nest correct?
Could the action in the data package run the template(s) and then add these variables as part of the action?
I am all for simple! I guess I thought the purpose of nest using pharmaverseadam was a one-stop shop for the ADaMs they need. So a user could grab the...
Reflecting on discussion - it seems this issue should be closed. Please re-open if not
Just musing on this more - a user shouldn't really be calling non-exported objects from a package. Feels like there is a reason a dev of package didn't want to...
Could this be a separate function that is called after derive_vars_joined - perhaps a generic cumulative type function could be created that isn't tightly tied derive_vars_joined so keep the modular...