Differences between revisions 68 and 69
Revision 68 as of 2018-08-29 10:29:25
Size: 5391
Editor: GabiRoeger
Comment:
Revision 69 as of 2018-09-21 12:38:21
Size: 5451
Comment: Lots of reviews done during the sprint
Deletions are marked like this. Additions are marked like this.
Line 37: Line 37:
 * https://bitbucket.org/malte/autonice/pull-requests/1: Python autonice (code review, Jendrik)
 * http://issues.fast-downward.org/issue749: allow putting plugins into sections in the documentation (code review, Jendrik)
 * --(https://bitbucket.org/malte/autonice/pull-requests/1: Python autonice (code review, Jendrik))--
 * --(http://issues.fast-downward.org/issue749: allow putting plugins into sections in the documentation (code review, Jendrik))--
Line 40: Line 40:
 * http://issues.fast-downward.org/issue766: convert Heuristic options to Evaluator (code review, Manuel)
 * http://issues.fast-downward.org/issue774: move axiom evaluator and state packer into task interface (code review, Florian)
 * http://issues.fast-downward.org/issue198: issues when using multiple heuristics (code review, Salomé)
 * --(http://issues.fast-downward.org/issue766: convert Heuristic options to Evaluator (code review, Manuel))--
 * --(http://issues.fast-downward.org/issue774: move axiom evaluator and state packer into task interface (code review, Florian))--
 * --(http://issues.fast-downward.org/issue198: issues when using multiple heuristics (code review, Salomé))--
Line 44: Line 44:
 * http://issues.fast-downward.org/issue790: fix various minor problems in headers (code review, Jendrik)  * --(http://issues.fast-downward.org/issue790: fix various minor problems in headers (code review, Jendrik))--
Line 46: Line 46:
 * http://issues.fast-downward.org/issue791: Use task interface for StateRegistry (code review, Florian)
 * http://issues.fast-downward.org/issue792: Get rid of g_successor_generator (code review, Florian)
 * http://issues.fast-downward.org/issue793: Introduce/collect utility methods for printing proxy objects (code review, Florian)
 * http://issues.fast-downward.org/issue794: Reduce code duplication in axiom evaluator (code review, Florian)
 * --(http://issues.fast-downward.org/issue791: Use task interface for StateRegistry (code review, Florian))--
 * --(http://issues.fast-downward.org/issue792: Get rid of g_successor_generator (code review, Florian))--
 * --(http://issues.fast-downward.org/issue793: Introduce/collect utility methods for printing proxy objects (code review, Florian))--
 * --(http://issues.fast-downward.org/issue794: Reduce code duplication in axiom evaluator (code review, Florian))--

Review queue

Why a queue and how to use it?

  • The queue is intended as a TODO list for Malte. We hope that it can be a useful tool for Malte to keep track of who wants want and to keep the waiting times reasonably fair.
  • Use the queue for code reviews, discussions and other Fast Downward things that Malte should work on.
  • As of now, the queue is intended to be used by the Basel AI group only. If you think this should be changed, please get in touch.
  • Add new items to the end of the queue. Usually, Malte will work from the front of the queue, but there may be reasons to promote/demote things.
  • There is no need to put everything into the queue. It is intended for things that you would ideally like to get done right now, but that are not terribly urgent.

  • The queue should be easy to use and useful for everyone involved. If it turns out that this is not the case, we should get rid of it again.

The queue

FastDownward: ForDevelopers/ReviewQueue (last edited 2023-07-28 09:55:50 by MalteHelmert)