= How to make good tickets? = [[TOC()]] == What is a ticket? == Tickets are '''units of work''' that we agree upon in the team. Each team member is responsible for maintaining their own tickets. Log in and browse to you personal tickets at {5} Tickets are essential tools to help us run our work meetings, communicate ideas and work together. Also we use them to work together (multiple people can participate in one ticket). Below some pointers: == What makes a good ticket? == For a good ticket you can easily answer the following questions: When the ticket is new: * Motivation: '''Why or for whom''' do we have this ticket? For example: for LifeLines. * Scope: What is the '''expected output''' to close this ticket? For example: wiki page, Excel file, Working demo, Unit test, etc. When the ticket is worked upon: * Actions: What are we '''planning to do to''' produce the expected output? For example: interview, program, design, statistical test, etc. * Evaluation: What are the '''criteria of succes''' on which we (or the customer) decides the outcome is good enough? For example: evaluated by person X, test-case passed, presented at meeting X. Please use the discussion mechanism to have discussions on the actions. It is like a forum :-) == Size of tickets? == Tickets should be not be to big or too small; * about THREE tickets per personweek is enough. * If you can't answer the questions above in 15 mins your ticket is probably too big! * If you can answer them in 1 min your ticket is too small. * Please bundle small todos into ONE ticket so we can still discuss them during workmeeting. For example: '''Design a user interface for LIMS''' is way too big. Expected evolution of your tickets in such case could be: * Ticket 1: '''Create a mock-up for lab data-entry based on protocols'''. [[BR]]Basically you now make answering Q1 and Q2 a ticket :-) Expected output is a wiki page with the mock-up that has been evaluated with Morris and/or team members. * Ticket 2: '''Implement user interface for lab data-entry based on protocols'''.[[BR]] Expected output is a working demo of (link to output of ticket 1 here) + manual that has been evaluated with Morris and/or team members. * Ticket 3: '''User test interface for lab-entry'''. [[BR]] Expected output is a log on where the user had trouble. Typical result may be that Ticket 2 is reopened for improvement.. Implicit in above is: the code is commited to SVN and there is a manual on the wiki of your component. == How to use Milestones? == We use '''milestones''' to organize our tickets in time. Tickets can move from one milestone to another if they are not yet closed. You always should take a close look to three milestones: * This weeks * Next week * Scheduled Next (default) This you can easily see these if you go to report {5}. Every week each team member makes sure this list is up to date. And we walk through them to see what has been achieved in that week. == What ticket type to use? == Currently we have four flavors of tickets. For each of them examples are shown: |---------------- {{{#!td valign="top" todo }}} {{{#!td valign="top" A '''todo''' is a unit of work that does not result in commits to SVN but instead in a wiki page, Excel document, a presentation and so on. So what you need to do is describe this expected output. Example ticket: ||title: ||Comparison of imputation methods Beagle, Impute and MACH || ||component: ||gwas || ||description: ||Why or for whom?[[BR]]For medgen and bbmri we want to compare imputation methods Beagle, Impute and MACH.[[BR]][[BR]]Expected output?[[BR]]An Excel document that describes a comparable percentage correctly estimated SNPs.[[BR]][[BR]]Action plan?[[BR]]Take existing genotype data from 1M arrays and split the data randomly in two. The first half we input into the impute methods. Then we compare the SNPs called with those actually called on the array and calculate percentage correctly estimated SNPs.[[BR]][[BR]]Evaluation plan?[[BR]]We will feed back results to Lude and Cisca's teams to evaluate outcomes and if necessary propose further testing. || }}} |---------------- {{{#!td valign="top" enhancement }}} {{{#!td valign="top" An '''enhancement''' is working on a new set of features in our software. In this case we EXPECT that there has already been a ticket that defines this enhancement. In these cases the ticket is simple: Example ticket: ||title: ||Implement use case: SearchQTLprofiles || ||component: ||xqtl || ||description: ||Implement the enhancement as described in http://www.xgap.org/wiki/XqtlReview#Usecase1.SearchQTLprofiles || }}} |---------------- {{{#!td valign="top" defect }}} {{{#!td valign="top" A '''defect''' is a known error in our code base. In this case the ticket MUST describe what you need to do to reproduce the defect and you MUST create a test that shows the problem. If possible you will first create a UnitTest that shows the bug so that it can never come back without us knowing. Example ticket: ||title: ||Fix JPA if primary key is not named 'id' || ||component: ||molgenis || ||description: ||Why or for whom?[[BR]]This ticket is a showstopper for ticket xyz.[[BR]][[BR]]Expected output?[[BR]]1. checkout molgenis and molgenis_test[[BR]]2. add a new with a [[BR]]2. generate code[[BR]]3. run build.xml without errors[[BR]][[BR]]Action plan?[[BR]]Correct generators to get rid of compile errors.[[BR]][[BR]]Evaluation method? UnitTest. If it is a user interface error you have to describe the procedure here. || }}} |---------------- {{{#!td valign="top" test }}} {{{#!td valign="top" A '''test''' is a special ticket to move the todo of 'testing' an enhancement or bugfix to others in the team. Usually it only refers to the other ticket where the actual bug/enhancement is described. Also newly reported bugs may first need a test case before the tickets can be changed into a bugfix. Example ticket: ||title: ||Java REST client doesn't work || ||component: ||molgenis || ||description: ||Why or for whom?[[BR]]We got a report stating that the REST api doesn't work from Java client[[BR]][[BR]]Expected output?[[BR]]Have a test case that uses the REST api and succeeded/fails[[BR]][[BR]]Action plan?[[BR]]Create a small REST client[[BR]][[BR]]Evaluation method?[[BR]]Feed back to the bug reporter what we found and what actions we will take.|| }}}