Minutes 101210: Difference between revisions

From KitwarePublic
Jump to navigationJump to search
No edit summary
 
(20 intermediate revisions by 2 users not shown)
Line 1: Line 1:
== Attendees ==
== Attendees ==


* Bill Lorensen
* Alex Gouaillard
* Ziv Zaniv
* Ziv Zaniv
* Xiaoxiao Liu
* Xiaoxiao Liu
* Wes Turner
* Wes Turner
* Jim Miller
* Jim Miller
* Brad Lowekamp
* Hans Johnson
* Hans Johnson
* Vincent Magnotta
* Vincent Magnotta
Line 10: Line 13:
* Sean Megason
* Sean Megason
* John Galeotti
* John Galeotti
* Mark Roden
* Mathieu Malaterre
* Cory Quammen


== Project Topics ==
== Project Topics ==
Line 15: Line 21:
* Reviewed plans for the Iowa meeting  
* Reviewed plans for the Iowa meeting  
** Retouched the Agenda
** Retouched the Agenda
* Task order #1 (modifications ?)


== Technical Topics ==
== Technical Topics ==


* Licensing
* Licensing
* Dashboard review
* Gerrit workflow questions
** How to manage patches that were modified ?
** Should a second patch require full review again ?
** Problem: the Gerrit comments should be kept.
*** but the reviewer's scores are not kept for subsequent patches.
** What are the rules for Gerrit to consolidate review rates ?
*** Particularly the +2 versus -2.
**** We would prefer that negative reviews override positive ones.
* Schedule regular Gerrit review sessions.
** Add an Expiration date to entries in Gerrit.
* Hans Johnson: Gerrit change proposal:
** Add a list of the reviewers names
** (at least in the Wiki...)
* Update the documentation.
** Doxygen


= Action Items =
= Action Items =
* Luis / Kitware to look at windows machines issues with shared libraries openjpeg
* Marcus / look at Gerrit customization of the rule for consolidating review scores.
** See if negative reviews can take precedence over positive ones.
** See if the list of reviewers can be displayed to the submitters.
* Brad D. to complete the workflow guide for Git & Gerrit.
* Hans Johnson will update the copyright headers of all ITK files.
* Luis to generate a Weekly build of the Doxygen documentation.

Latest revision as of 01:10, 13 October 2010

Attendees

  • Bill Lorensen
  • Alex Gouaillard
  • Ziv Zaniv
  • Xiaoxiao Liu
  • Wes Turner
  • Jim Miller
  • Brad Lowekamp
  • Hans Johnson
  • Vincent Magnotta
  • Luis Ibanez
  • Sean Megason
  • John Galeotti
  • Mark Roden
  • Mathieu Malaterre
  • Cory Quammen

Project Topics

  • Reviewed plans for the Iowa meeting
    • Retouched the Agenda
  • Task order #1 (modifications ?)

Technical Topics

  • Licensing
  • Dashboard review
  • Gerrit workflow questions
    • How to manage patches that were modified ?
    • Should a second patch require full review again ?
    • Problem: the Gerrit comments should be kept.
      • but the reviewer's scores are not kept for subsequent patches.
    • What are the rules for Gerrit to consolidate review rates ?
      • Particularly the +2 versus -2.
        • We would prefer that negative reviews override positive ones.
  • Schedule regular Gerrit review sessions.
    • Add an Expiration date to entries in Gerrit.
  • Hans Johnson: Gerrit change proposal:
    • Add a list of the reviewers names
    • (at least in the Wiki...)
  • Update the documentation.
    • Doxygen

Action Items

  • Luis / Kitware to look at windows machines issues with shared libraries openjpeg
  • Marcus / look at Gerrit customization of the rule for consolidating review scores.
    • See if negative reviews can take precedence over positive ones.
    • See if the list of reviewers can be displayed to the submitters.
  • Brad D. to complete the workflow guide for Git & Gerrit.
  • Hans Johnson will update the copyright headers of all ITK files.
  • Luis to generate a Weekly build of the Doxygen documentation.