We just had our first meeting online. Thanks to all that participated: @syclik, @bbbales2, @sakrejda, @roualdes, @anon79882417, @Bob_Carpenter, @seantalts, @stevebronder, @yizhang, @increasechief.
There was a log of good feedback. Here’s what we decided to do:
- Over the next month, put together a standard for reviewing pull requests. This discourse thread is a starting point: Standard for reviewing pull requests in the Math library
- This will include:
- how to review a pull request correctly
- who should review a pull request
- @seantalts, mind filling in whatever questions you have that I didn’t write down?
- This will go on the Math wiki.
- We should encourage more people to review code. We can do this by splitting apart reviewer tasks across different people since a lot of pull requests are multi-faceted.
- This will include:
- We discussed the next release of Math and Stan. We will release Math v2.18.0 and Stan v2.18.0 when CmdStan has MPI ready to go.
- For the 1-d integrator pull request, we need to go through and see if a decision has been made regarding Boost. If it hasn’t, we need to clearly come up with a decision. If it has, we need that to be clear.
We currently have:
- 149 open issues
- 12 open pull requests
- only 7 pass tests
Some other things we need to do:
- prioritize what we should work on for the Math library
- we need criteria for making decisions
- clarify decisions when they’re made
We can pick those up following the next meeting.