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 priors for all parameters and make defining these priors more flexible #394

Open
acope3 opened this issue Aug 14, 2023 · 2 comments
Open
Assignees

Comments

@acope3
Copy link
Owner

acope3 commented Aug 14, 2023

Currently, some of the parameters ($\Delta\eta$, $s_{\phi}$) in our models have improper priors (i.e. a uniform defined on $(-\infty,\infty)$). I would like to ensure all models' parameters have proper priors and there is some flexibility in how these are implemented. The prior for $\Delta\mathit{M}$ is a good example.

Here are my thoughts,

  1. $\Delta\eta$ (ROC), $\Delta\omega$ (FONSE) - Normal distribution prior (same as $\Delta\mathit{M}$)
  2. $s_{\phi}$ - Half-Cauchy
  3. $\alpha$ (PA, PANSE) - uniform, Half-Cauchy
  4. $\lambda$ (PANSE) - uniform, Half-Cauchy
  5. $Z/Y$ (PANSE) - Half-Cauchy (Note: Y is the total number of reads here, but should make values closer to 0)
  6. $\lambda'$ (PA) - uniform, Half-Cauchy
  7. NSE Rates (PANSE) -- uniform (log-scale), uniform (natural scale), exponential distribution
@acope3 acope3 self-assigned this Aug 14, 2023
@mikegilchrist
Copy link
Contributor

mikegilchrist commented Aug 15, 2023

In general, I suggest we follow the suggestions from the stan team and point you to (https://github.com/stan-dev/stan/wiki/Prior-Choice-Recommendations). My reading is that they recommend using a gamma rather than a 1/2 Cauchy (which is always maximal at 0) for variables bounded to the positive reals. If my understanding is correct, I'd recommend using the gamma instead of the 1/2 cauchy or exponential. On a more minor point, I recommend a multivariate normal for codon specific parameters ($\Delta M, \Delta \eta$), as I recall $\Delta \omega$ is an odds ratio, so we might want to treat that one differently.

Also to be clear, most of the parameters constrained to be positive are proposed on the log scale. I'm assuming, however, that these priors will be implemented on the natural scale. Is this assumption correct?

Finally, is this the smartest approach? Might we be better off to implement the models using rstan and brms which can autogenerate stan code for logistic and other forms of regression?

@mikegilchrist
Copy link
Contributor

I have implemented (but not rigourously tested) allowing us to add priors to to dEta in essentially the same way they are implemented for dM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants