Skip to content
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

Add an application vignette #448

Open
2 tasks
danielinteractive opened this issue Jun 27, 2024 · 7 comments
Open
2 tasks

Add an application vignette #448

danielinteractive opened this issue Jun 27, 2024 · 7 comments
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers

Comments

@danielinteractive
Copy link
Collaborator

To do:

  • Add a new vignette showcasing a practical application of mmrm
  • Include usage of emmeans (going beyond what we have already in the introduction vignette)
@danielinteractive danielinteractive added good first issue Good for newcomers documentation Improvements or additions to documentation labels Jun 27, 2024
@danielinteractive
Copy link
Collaborator Author

@yonicd I finally added this issue, feel free to add other ideas in the description

@yonicd
Copy link
Collaborator

yonicd commented Jul 4, 2024

Could we run a cLDA ref 1 ref 2 using {mmrm}? that could be a nice usage vignette.

@danielinteractive
Copy link
Collaborator Author

Not sure, could be worth a try! I think we had a similar idea (cLDA) in the issues/discussions before

@yonicd
Copy link
Collaborator

yonicd commented Jul 4, 2024

Ref2 has a nice example w gls, it is straight fw. It looks like you need to setup the design matrix a bit manually, but overall easy enough.

@yonicd
Copy link
Collaborator

yonicd commented Jul 11, 2024

i tested out the cLDA formula yesterday.

It is pretty simple.

  1. pivot longer fev_data to include baseline in the FEV and add a VISITN=0
  2. define the formula as FEV ~ VISITN + VISITN:ARMCD + us(VISITF|USUBJID)

This notation is analogous to a design matrix that does not include the ARMCD as a variable but does include VISITN and the interaction of VISITN*ARMCD.

This design matrix aligns with the model assumption:

In cLDA, the group means are assumed to be equal at baseline (hence the “constrained” in cLDA), which is a reasonable assumption in a randomized trial. As described below, this strategy is an important exception of the rule that an interaction should not be included in a model without its main effects.

One of the motivations of this design is when there is missingness in the baseline, in such cases in the standard mmrm formula those rows would be dropped since FEV_BL is missing.

@danielinteractive I think then maybe it would be worth to have a version of data with missingness in baseline to show how mmrm and clda could yield different inferential results, what do you think?

@danielinteractive
Copy link
Collaborator Author

Thanks @yonicd for digging into this, much appreciated!

Yeah, I think that modified version could be created inside the vignette to demonstrate this.

@danielinteractive
Copy link
Collaborator Author

@yonicd would you like to take this on in terms of adding a vignette?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers
Projects
Status: Issue To start
Development

No branches or pull requests

2 participants