Help clean up Stan.

24 views
Skip to first unread message

Daniel Lee

unread,
Mar 17, 2015, 11:17:35 AM3/17/15
to stan...@googlegroups.com
All devs,

Can you:

1) get rid of old branches.
Go to: https://github.com/stan-dev/stan/branches/yours
For anything you don't need, click the red trash can (last icon on
the right).
For the other branches, if you need help getting it up to the
current develop state, please let me or the group know.

2) help clean up existing issues.
Our open issues can be found here: https://github.com/stan-dev/stan/issues
If it's already been fixed, please comment that it has been fixed
and close the issue.
If it can't be fixed, indicate that in a comment and close it.
If it can be fixed, please write down some notes on how to
implement it and either:
a) implement a fix and create a pull request
b) leave notes on how it should be fixed (don't just say it can
be fixed, but rather what could be done to fix it -- it'll make it
easier for someone that may need just a little push to fixing it.)
c) optional: assign it to someone you think can fix it.


We currently have 52 branches (2 are necessary: develop and master).
And we have 253 open issues.


Daniel

Michael Betancourt

unread,
Mar 17, 2015, 11:54:29 AM3/17/15
to stan...@googlegroups.com

> 1) get rid of old branches.
> Go to: https://github.com/stan-dev/stan/branches/yours
> For anything you don't need, click the red trash can (last icon on
> the right).
> For the other branches, if you need help getting it up to the
> current develop state, please let me or the group know.

Just a few notes on some of my branches:

feature/trajectory_diagnostic
Will need to be rewritten after the command refactor but I’m keeping this
here to motivate the rewrite.

feature/rhmc
Waiting for higher-order autodiff.

feature/stat_valid_test
Needs command to be refactored so it can run chains directly from the C++.
Technically possible to do now but I’d rather wait and do it right.

feature/metropolis
This was a simple algorithm to code up the the adaptation is a nightmare
and Peter could never get it to work well enough to warrant inclusion.
We should either add without adaptation (perhaps with a big warning
message to users) or kill the branch as it’ll be easy to rewrite later.

cmdstan-issue-21-elapsed-time-duplicated
I believe this was fixed, no?

feature/issue-462-set-mass-matrix
Another branch that will need to be rewritten but the code here has
some useful spec so I’d prefer to keep it until then.

Daniel Lee

unread,
Mar 17, 2015, 12:32:31 PM3/17/15
to stan...@googlegroups.com
There was one more thing I forgot to mention. Go and review pull requests too.
https://github.com/stan-dev/stan/pulls

If it's ok, mark it with the label "code review complete." If there
are any issues with the pull request, comment on it and don't mark it
as that -- maybe I should change that to "code review passed" or
something.



Daniel
> --
> You received this message because you are subscribed to the Google Groups "stan development mailing list" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to stan-dev+u...@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
Reply all
Reply to author
Forward
0 new messages