Retrieved dropout vignette

[nociale]

Solve #414

[github-actions]

✅ All contributors have signed the CLA
_{Posted by the CLA Assistant Lite bot.}

[nociale]

I have read the CLA Document and I hereby sign the CLA

[gowerc]

Hey @nociale , Apologies for what is a super trivial point but I just thought I'd highlight that in general it is recommended to split out long markdown lines across multiple lines. This is mostly because (a) the line then first nicer onto the screen if not using text-wrapping and (b) because git works per line so is easier for it to show diffs if you need to modify the text in the future e.g.

- The probability of the intercurrent event study drug discontinuation after each visit is calculated according to a logistic model which depends on the observed outcome at that visit. Specifically, a visit-wise discontinuation probability of 3% and 4% in the control and intervention group, respectively, is specified in case the observed outcome is equal to 50 (the mean value at baseline). The odds of a discontinuation is simulated to increase by +10% for each +1 point increase of the observed outcome.

could be:

- The probability of the intercurrent event study drug discontinuation after each visit is calculated according to a logistic model which depends on the observed outcome at that visit.
Specifically, a visit-wise discontinuation probability of 3% and 4% in the control and intervention group, respectively, is specified in case the observed outcome is equal to 50 (the mean value at baseline).
The odds of a discontinuation is simulated to increase by +10% for each +1 point increase of the observed outcome.

I should add that Markdown ignores single newlines. That is when the text is rendered to html single \n are removed so the text will still appear on a single line, you have to add a double space or double newlines \n\n to get the text to actually appear on a new line

But yer this is a super trivial comment and can be ignored if you don't have the time/energy to update the code.

[wolbersm]

I have read the CLA Document and I hereby sign the CLA

[nociale]

Hi @gowerc , thanks for the suggestion, I tried to apply it. Could you please check if everything is fine and we can merge?

[gowerc]

@nociale - If you are happy with all your proposed changes feel free to commit them and resolve the conversations that you've addressed.

[nociale]

@gowerc I have tried to make the changes as discussed in the conversation and have re-generated the html vignette. Could you have a last look? Thanks!

[gowerc]

Some of the formatting was slightly broken in the rendered html so adjusted the Rmd very slightly to correct that, I also added the missing "asis" file and updated the build.R file to generate the vignette. I also added an entry to the NEWS.md file.

Otherwise looks ok to me will merge