Franzi2114

Results 35 comments of Franzi2114

Hey @spinkney, thanks for your reply! So when I understand you correctly, it should be no problem to hand in the paper and refer to my fork with an instruction...

> If you can provide the details below Up to now I have a fork of stan-dev/math: [email protected]:Franzi2114/math.git The function is on branch: feature/issue-2682-Add-7-parameter-DDM-PDF I did not change anything in...

Here you are @andrjohns: My stanc3 Fork is here: [email protected]:Franzi2114/stanc3.git and the branch has the same name as the other branch: feature/issue-2682-Add-7-parameter-DDM-PDF I just tested it locally. There it works...

Thanks @andrjohns! This was successful. I just tested it. I put the cmdstan-ddm-7pm.zip folder into my repo and updated the instruction on how to get started: [https://github.com/Franzi2114/math_HOW-TO-USE](https://github.com/Franzi2114/math_HOW-TO-USE)

> I'm not an expert in these but I'm curious how the 5 parameter one different than the `wiener_lpdf` already in stan? It's 4 parameters but then we separate in...

> add the `lcdfs` and the `rng` functions, Yeah, thanks @spinkney! That's true. When setting all variabilities to 0 and calling the function with `wiener_full_lpdf(a,v,w,t0,0,0,0)` the four parameter model will...

> Can you split this into three PRs rather than all together? Hi @andrjohns, thanks for your input! Yes, we can split our PR into three PRs. I will refer...

Hey @andrjohns, I am confused. I only merged the develop branch (where the new operands_and_partials_8 routine is integrated now) into this branch and the continuous-integration test fails at a point...

Hello together, now the two split-PRs (#2833, #2838) are merged into develop and into this branch and all tests run successfully. Who would review this PR? @andrjohns, @spinkney, @SteveBronder, @rok-cesnovar,...