Skip to content

Commit

Permalink
address review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
gowerc committed May 21, 2024
1 parent cea0d5e commit fbfc8de
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
6 changes: 4 additions & 2 deletions vignettes/custom-model.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@ knitr::opts_chunk$set(

This vignette shows a complete example for how to fit a custom longitudinal model. Note that full
details for the various different interfaces can be found in the "Extending jmpost" vignette.
This example implements the Wang, Sung et al. 2009 mixed exponential decay and linear growth model.
This example implements the Wang, Sung et al. 2009 mixed exponential decay and linear growth model
along with an exponential survival model.
In particular the following model will be implemented:

**Longitudinal Model**:
Expand Down Expand Up @@ -56,7 +57,8 @@ $$

Where:

- $\lambda_0$ is the baseline hazard function (e.g. we are fitting an exponential model)
- $\lambda_0$ is the baseline hazard rate. This is because for this
example we are using an exponential survival model e.g. $h_0(t) = \lambda_0$
- $t$ is the event time
- $G(.)$ is a link function that maps the subjects tumour growth parameters to a contribution to
the log-hazard function
Expand Down
2 changes: 1 addition & 1 deletion vignettes/extending-jmpost.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ Unlike the survival distributions, the longitudinal models are a lot more flexib
constraints on how they are implemented. That is there aren't any specific variables or functions
that you need to define.

That being said there are a several optional features of `jmpost` that do require the use specific
That being said there are a several optional features of `jmpost` that do require the use of specific
interfaces if you want to enable them for your model.

### 1) `loo` integration
Expand Down

0 comments on commit fbfc8de

Please sign in to comment.