| 1 | = How to make good tickets? = |
| 2 | [[TOC()]] |
| 3 | |
| 4 | == What is a ticket == |
| 5 | |
| 6 | Tickets are units of worked that we agree upon in the team. |
| 7 | |
| 8 | We use them to organize our work meetings and communicate our ideas amongst each other. Therefore: |
| 9 | * tickets should be understandeable, see below. |
| 10 | * tickets should be not be to big or too small; about three tickets per personweek is enough |
| 11 | * tickets are often shared between multiple people |
| 12 | * if you have many small todos, please put them in ONE extra ticket so we can still discuss them during work meeting |
| 13 | |
| 14 | == What makes a good ticket == |
| 15 | |
| 16 | '''Tickets always have a description''' that explains for all team members to understand: |
| 17 | |
| 18 | * Motivation: why or for whom do we do this? For example? |
| 19 | * Expected output: what will we have in hand when the tickets is closed? |
| 20 | * (Optional) what is the plan of '''actions''' to achieve it. |
| 21 | * (Optional) how do we measure this output. For example a 'test case'. |
| 22 | |
| 23 | Please use the discussion mechanism to log your progress on the ticket. |
| 24 | |
| 25 | For example: |
| 26 | For medgen and bbmri (=for whom): we compare imputation methods Beagle, Impute and MACH to the percentage correctly estimated SNPs (=output). We 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 calculate percentage correctly estimated SNPs on the other half for each method (=actions). |
| 27 | |
| 28 | |
| 29 | |
| 30 | == Ticket types == |
| 31 | |
| 32 | Currently we have four flavors of tickets: |
| 33 | |
| 34 | === defect === |
| 35 | A defect is a known error in our code base. Therefore the ticket MUST describe what you need to do to reproduce the defect. |
| 36 | ExpeIf possible you will first create a UnitTest that shows the bug so that it can never come back without us knowing. |
| 37 | |
| 38 | {{{ |
| 39 | To reproduce the defect: |
| 40 | 1. checkout molgenis and molgenis_test |
| 41 | 2. generate code |
| 42 | 3. run handwritten/tests/TestThisBug.java |
| 43 | |
| 44 | The error log is as follows: |
| 45 | <paste of error log> |
| 46 | |
| 47 | }}} |
| 48 | |
| 49 | === Enhancement === |
| 50 | |
| 51 | |
| 52 | '''todo''' - these are actions that don't produce code but something else. For example: an excel with quality scores for each NGS lane. |
| 53 | |
| 54 | |