stan-dev / stan

Stan development repository. The master branch contains the current release. The develop branch contains the latest stable development. See the Developer Process Wiki for details.
https://mc-stan.org
BSD 3-Clause "New" or "Revised" License
2.6k stars 370 forks source link

Add "Ratcliff" log-likelihood, Wiener process with intrinsic trial-to-trial drift variability #1875

Closed mdnunez closed 4 years ago

mdnunez commented 8 years ago

New Feature Suggestion:

Add a Wiener likelihood function with intrinsic trial to trial variability in the drift rate, titled the "Ratcliff" likelihood function

Description:

Adding a parameter that describes the variance in drift rate over trials to the Wiener likelihood function yields better estimates of incorrect reaction time (see Table 3 of Nunez et al, 2016 for an example deficit of incorrect-RT prediction with a hierarchical version of the simple Wiener likelihood). This version of the diffusion model and its extensions was popularized by Ratcliff (1978; see also Ratcliff and McKoon, 2008) and is used widely in the mathematical psychology and neuroscience communities. Because of sampling techniques developed by Navarro and Fuss (2009) and probability density function derivations by Tuerlinckx (2004), it is relatively straight-forward to copy and change the existing Wiener likelihood sampler in Stan to include intrinsic trial-to-trial variability. This method has already been implemented by existing hierarchical "drift-diffusion" model samplers (Wiecki, 2003).

Additional Information:

I have a version of "ratcliff_log.hpp" code mostly finished however it needs to be tested and compared to simulations. Any helpful pointers to what needs to be changed in other parts of Stan code would be appreciated.

Citations:

v2.9.0

bob-carpenter commented 8 years ago

Any helpful pointers to what needs to be changed in other parts of Stan code would be appreciated.

See:

mdnunez commented 8 years ago

This issue was moved to stan-dev/math#287

mcol commented 4 years ago

Already in math.

tiagocabaco commented 4 years ago

Thanks looking into this.

I don't see it however where this distribution is implemented in the Math library? Is it in a different branch?

Thanks again, Tiago Tiago Cabaço

On Fri, Feb 21, 2020 at 7:20 PM Marco Colombo notifications@github.com wrote:

Already in math.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/stan-dev/stan/issues/1875?email_source=notifications&email_token=ABYMCKPRJKUXVUOYBX4UCLLREALO3A5CNFSM4CBM5CR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMTTPWA#issuecomment-589772760, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYMCKKV6WJCUA4QQMVEQ4LREALO3ANCNFSM4CBM5CRQ .

mcol commented 4 years ago

It's not implemented yet, but we already have issue stan-dev/math#287 to track the work.

tiagocabaco commented 4 years ago

Thank you for the clarification.

Best regards Tiago Cabaço

On Wed, Feb 26, 2020 at 2:29 PM Marco Colombo notifications@github.com wrote:

It's not implemented yet, but we already have issue stan-dev/math#287 https://github.com/stan-dev/math/issues/287 to track the work.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/stan-dev/stan/issues/1875?email_source=notifications&email_token=ABYMCKJJ73LXP4MAXPWYD3TREZVE7A5CNFSM4CBM5CR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENAHETA#issuecomment-591426124, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYMCKL42BCZEAMRNEUANS3REZVE7ANCNFSM4CBM5CRQ .