Franzi2114
Franzi2114
> I think at the point we have 8 edges it might be good to turn the impl into having an underlying `std::tuple` Yeah, maybe that makes sense. But I...
Hi, I have two more questions: > with reasonable defaults @bob-carpenter, how is the way to define defaults for our new function? Is it related to the Stan_math_signatures.ml or is...
Hi @bob-carpenter , thanks for your reply! > then submit a PR from your branch. When you do that, we'll be able to import your new branch for code review....
The PR is in progress! (#2822) The code can be found in my [fork](https://github.com/Franzi2114/math/tree/feature/issue-2682-Add-7-parameter-DDM-PDF). An instruction how to integrate this stan version into a local environment can be found [here](https://github.com/Franzi2114/math_HOW-TO-USE).
Hey @bob-carpenter, now the PR for this issue is open (#2822) and all tests run successfully. I have some questions to the next steps: Who starts the workflows and who...
Hey @bob-carpenter, I already tried to find a person but unfortunately this person was familiar with the function but not with the Stan source code. I checked the Stan developers...
> Then I can start reviewing the code. Hi @charlesm93, thanks for your reply! I also saw that @spinkney started reviewing the contribution. (Thanks for that @spinkney!!) I think he...
Hey guys (@charlesm93, @spinkney), unfortunately I'm not available next week. Additionally, the feature-freeze for the next Stan-version is on November, 7th. Do you think it is realistic to have the...
Hey @charlesm93 and @andrjohns, do you also have time tomorrow at 2pm for a meeting or shall we postpone it?
Hi @charlesm93, hi @spinkney, hi @valentin12, how about a meeting this or next week to talk about the PR for the 7-parameter diffusion model? I am currently working on a...