Skip to content

DDM task wrapped from DiffModel.jl. #9

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 105 commits into
base: main
Choose a base branch
from
Draft

DDM task wrapped from DiffModel.jl. #9

wants to merge 105 commits into from

Conversation

janfb
Copy link
Contributor

@janfb janfb commented Apr 26, 2021 •
edited
Loading

basic DDM task with only two parameters such that data and likelihoods can be computed using DiffModel.jl.

TODO:

  • implement reference posterior sampling
  • remove local paths
  • add references

@janfb janfb marked this pull request as ready for review April 27, 2021 13:09
@janfb
Copy link
Contributor Author

janfb commented Apr 27, 2021

@jan-matthis could you have a look please? especially at the last three two commits?

rdgao
rdgao reviewed Apr 30, 2021
View reviewed changes
@jan-matthis
Copy link
Contributor

Merging #10 brought about some conflicts, see above

janfb and others added 25 commits May 19, 2021 12:17
@janfb
add DDM task wrapped from DiffModel.jl.
5e3ba87
@janfb
move paramter batch for loop to julia, add potential fun.
091b971
@janfb
reference posterior sampling for ddm.;
77a2bcf
@janfb
adapt mcmc to work with either potential function or model.
7af30fb
@janfb
refactor log to logprob, loop over parameter batch.
d9fe569
@janfb
add transform, get posterior samples from grid.
1c3f428
@janfb
formatting
2694a5b
@janfb
fix summation bug and get reference via mcmc
1f4c276
@janfb
introduce 4 param version via asymmetric bounds.
386d0ae
@janfb
change to 3 parameters, removing ndt.
409dc8d
@janfb
add seeding to Julia wrapper.
b91e54c
@janfb
add seeding, add repeated seeds to compare refs.
b0b8979
@rdgao @janfb
added analogous python functions for ddm simulator and likelihood usi…
37d4eab 
…ng navarro-fuss approximation
@janfb
add seeding, add repeated seeds to compare refs.
e4d5929
@janfb
add refs for 5 obs, 4 ref-methods.
3afdc55
@janfb
custom _setup for ddm to allow seeds with different num_trials.
34864e6
@janfb
references for 5 observations with 4 increasing num_trials.
88003d9
@janfb
add mcmc reference for 10 obs, 3d, 1024 trials.
a52db2a
@janfb
change references to single trial.
8d4e498
@janfb
change reference to obs 1 for 4 different #trials.
5fdd646
@janfb
encode down choices as negative rts.
b04b449
@janfb
adapt to new rt encoding, add hyperparams.
c6e8e2e
@janfb
fix pairplot imports.
b52ef02
@janfb
add moments metrics.
34c7148
@janfb
add references with new rts coding.
911ec20
@janfb janfb force-pushed the ddm-task branch 10 times, most recently from 434ada2 to 3ac9597 Compare February 14, 2023 14:48
@jan-matthis jan-matthis added the enhancement New feature or request label Feb 18, 2023
@janfb janfb force-pushed the ddm-task branch 2 times, most recently from c3e5b9f to 0c1cd80 Compare February 20, 2023 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@rdgao rdgao rdgao left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
enhancement New feature or request
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@janfb @jan-matthis @rdgao