Franzi2114

Results 35 comments of 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...