/b-gtk/fix-viz

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/b-gtk/fix-viz

« back to all changes in this revision

Viewing changes to viz/linegraph.py

  • Committer: Gary van der Merwe
  • Date: 2007-08-31 11:24:56 UTC
  • mto: (256.2.37 gtk)
  • mto: This revision was merged to the branch mainline in revision 289.
  • Revision ID: garyvdm@gmail.com-20070831112456-x0if1u8gsbm04tbf
Show Children

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# -*- coding: UTF-8 -*-
2
 
"""Directed graph production.
3
 
 
4
 
This module contains the code to produce an ordered directed graph of a
5
 
bzr branch, such as we display in the tree view at the top of the bzrk
6
 
window.
7
 
"""
8
 
 
9
 
__copyright__ = "Copyright © 2005 Canonical Ltd."
10
 
__author__    = "Scott James Remnant <scott@ubuntu.com>"
11
 
 
12
 
from bzrlib.revision import NULL_REVISION
13
 
from bzrlib.tsort import merge_sort
14
 
 
15
 
def linegraph(repository, start_revs, maxnum, broken_line_length = None,
16
 
              graph_data = True, mainline_only = False):
17
 
    """Produce a directed graph of a bzr repository.
18
 
 
19
 
    Returns a tuple of (line_graph, revid_index, columns_len) where
20
 
    * line_graph is a list of tuples of (revid,
21
 
                                         node,
22
 
                                         lines,
23
 
                                         parents,
24
 
                                         children,
25
 
                                         revno_sequence),
26
 
    * revid_index is a dict of each revision with the key being the revid, and
27
 
      the value the row index, and
28
 
    * columns_len is the number of columns need to draw the line graph.
29
 
    
30
 
 
31
 
    Node is a tuple of (column, colour) with column being a zero-indexed
32
 
    column number of the graph that this revision represents and colour
33
 
    being a zero-indexed colour (which doesn't specify any actual colour
34
 
    in particular) to draw the node in.
35
 
 
36
 
    Lines is a list of tuples which represent lines you should draw away
37
 
    from the revision, if you also need to draw lines into the revision
38
 
    you should use the lines list from the previous iteration.  Each
39
 
    typle in the list is in the form (start, end, colour) with start and
40
 
    end being zero-indexed column numbers and colour as in node.
41
 
 
42
 
    It's up to you how to actually draw the nodes and lines (straight,
43
 
    curved, kinked, etc.) and to pick the actual colours for each index.
44
 
    """
45
 
    
46
 
    graph = repository.get_graph()
47
 
    graph_parents = {}
48
 
    ghosts = set()
49
 
    graph_children = {}
50
 
    for (revid, parent_revids) in graph.iter_ancestry(start_revs):
51
 
        if parent_revids is None:
52
 
            ghosts.add(revid)
53
 
            continue
54
 
        if parent_revids == (NULL_REVISION,):
55
 
            graph_parents[revid] = ()
56
 
        else:
57
 
            graph_parents[revid] = parent_revids
58
 
        for parent in parent_revids:
59
 
            graph_children.setdefault(parent, []).append(revid)
60
 
        graph_children.setdefault(revid, [])
61
 
    for ghost in ghosts:
62
 
        for ghost_child in graph_children[ghost]:
63
 
            graph_parents[ghost_child] = [p for p in graph_parents[ghost_child]
64
 
                                          if p not in ghosts]
65
 
    graph_parents["top:"] = start_revs
66
 
 
67
 
    if len(graph_parents)>0:
68
 
        merge_sorted_revisions = merge_sort(
69
 
            graph_parents,
70
 
            "top:",
71
 
            generate_revno=True)
72
 
    else:
73
 
        merge_sorted_revisions = ()
74
 
    
75
 
    if mainline_only:
76
 
        merge_sorted_revisions = [elem for elem in merge_sorted_revisions \
77
 
                                  if len(elem[3])==1 ]
78
 
 
79
 
    assert merge_sorted_revisions[0][1] == "top:"
80
 
    merge_sorted_revisions = merge_sorted_revisions[1:]
81
 
    
82
 
    revid_index = {}
83
 
    revno_index = {}
84
 
    
85
 
    # This will hold an item for each "branch". For a revisions, the revsion
86
 
    # number less the least significant digit is the branch_id, and used as the
87
 
    # key for the dict. Hence revision with the same revsion number less the
88
 
    # least significant digit are considered to be in the same branch line.
89
 
    # e.g.: for revisions 290.12.1 and 290.12.2, the branch_id would be 290.12,
90
 
    # and these two revisions will be in the same branch line. Each value is
91
 
    # a list of rev_indexes in the branch.
92
 
    branch_lines = {}
93
 
    
94
 
    linegraph = []    
95
 
    
96
 
    for (rev_index, (sequence_number,
97
 
                     revid,
98
 
                     merge_depth,
99
 
                     revno_sequence,
100
 
                     end_of_merge)) in enumerate(merge_sorted_revisions):
101
 
        if maxnum and rev_index >= maxnum:
102
 
            break
103
 
        revid_index[revid] = rev_index
104
 
        
105
 
        parents = graph_parents[revid]
106
 
        linegraph.append([revid,
107
 
                          None,
108
 
                          [],
109
 
                          parents,
110
 
                          None,
111
 
                          revno_sequence])
112
 
        
113
 
        if graph_data:
114
 
            revno_index[revno_sequence] = rev_index
115
 
            
116
 
            branch_id = revno_sequence[0:-1]
117
 
            
118
 
            branch_line = None
119
 
            if branch_id not in branch_lines:
120
 
                branch_line = []
121
 
                branch_lines[branch_id] = branch_line
122
 
            else:
123
 
                branch_line = branch_lines[branch_id]
124
 
            
125
 
            branch_line.append(rev_index)        
126
 
 
127
 
    if graph_data:
128
 
        branch_ids = branch_lines.keys()
129
 
    
130
 
        def branch_id_cmp(x, y):
131
 
            """Compaire branch_id's first by the number of digits, then reversed
132
 
            by their value"""
133
 
            len_x = len(x)
134
 
            len_y = len(y)
135
 
            if len_x == len_y:
136
 
                return -cmp(x, y)
137
 
            return cmp(len_x, len_y)
138
 
        
139
 
        branch_ids.sort(branch_id_cmp)
140
 
        # This will hold a tuple of (child_index, parent_index, col_index) for each
141
 
        # line that needs to be drawn. If col_index is not none, then the line is
142
 
        # drawn along that column, else the the line can be drawn directly between
143
 
        # the child and parent because either the child and parent are in the same
144
 
        # branch line, or the child and parent are 1 row apart.
145
 
        lines = []
146
 
        empty_column = [False for i in range(len(graph_parents))]
147
 
        # This will hold a bit map for each cell. If the cell is true, then the
148
 
        # cell allready contains a node or line. This use when deciding what column
149
 
        # to place a branch line or line in, without it overlaping something else.
150
 
        columns = [list(empty_column)]
151
 
        
152
 
        
153
 
        for branch_id in branch_ids:
154
 
            branch_line = branch_lines[branch_id]
155
 
            
156
 
            # Find the col_index for the direct parent branch. This will be the
157
 
            # starting point when looking for a free column.
158
 
            parent_col_index = 0
159
 
            parent_index = None
160
 
            if len(branch_id) > 1:
161
 
                parent_revno = branch_id[0:-1]
162
 
                if parent_revno in revno_index:
163
 
                    parent_index = revno_index[parent_revno]
164
 
                    parent_node = linegraph[parent_index][1]
165
 
                    if parent_node:
166
 
                        parent_col_index = parent_node[0]
167
 
                    
168
 
            
169
 
            col_search_order = _branch_line_col_search_order(columns,
170
 
                                                             parent_col_index)
171
 
            color = reduce(lambda x, y: x+y, branch_id, 0)
172
 
            cur_cont_line = []
173
 
            
174
 
            line_range = []
175
 
            last_rev_index = None
176
 
            for rev_index in branch_line:
177
 
                if last_rev_index:
178
 
                    if broken_line_length and \
179
 
                       rev_index - last_rev_index > broken_line_length:
180
 
                        line_range.append(last_rev_index+1)
181
 
                        line_range.append(rev_index-1)
182
 
                    else:
183
 
                        line_range.extend(range(last_rev_index+1, rev_index))
184
 
                
185
 
                line_range.append(rev_index)
186
 
                last_rev_index = rev_index
187
 
            
188
 
            if parent_index:
189
 
                if broken_line_length and \
190
 
                   parent_index - last_rev_index > broken_line_length:
191
 
                    line_range.append(last_rev_index+1)
192
 
                else:
193
 
                    line_range.extend(range(last_rev_index+1, parent_index))
194
 
            
195
 
            col_index = _find_free_column(columns,
196
 
                                          empty_column,
197
 
                                          col_search_order,
198
 
                                          line_range)
199
 
            node = (col_index, color)
200
 
            for rev_index in branch_line:
201
 
                linegraph[rev_index][1] = node
202
 
                columns[col_index][rev_index] = True
203
 
            
204
 
            for rev_index in branch_line:
205
 
                (sequence_number,
206
 
                     revid,
207
 
                     merge_depth,
208
 
                     revno_sequence,
209
 
                     end_of_merge) = merge_sorted_revisions[rev_index]
210
 
                
211
 
                linegraph[rev_index][4] = graph_children[revid]
212
 
                col_index = linegraph[rev_index][1][0]
213
 
                
214
 
                for parent_revid in graph_parents[revid]:
215
 
                    if parent_revid in revid_index:
216
 
                        
217
 
                        parent_index = revid_index[parent_revid]                            
218
 
                        parent_node = linegraph[parent_index][1]
219
 
                        if parent_node:
220
 
                            parent_col_index = parent_node[0]
221
 
                        else:
222
 
                            parent_col_index = None
223
 
                        col_search_order = \
224
 
                                _line_col_search_order(columns,
225
 
                                                       parent_col_index,
226
 
                                                       col_index)
227
 
                            
228
 
                        # If this line is really long, break it.
229
 
                        if len(branch_id) > 0 and \
230
 
                           broken_line_length and \
231
 
                           parent_index - rev_index > broken_line_length:
232
 
                            child_line_col_index = \
233
 
                                _find_free_column(columns,
234
 
                                                  empty_column,
235
 
                                                  col_search_order,
236
 
                                                  (rev_index + 1,))
237
 
                            _mark_column_as_used(columns,
238
 
                                                 child_line_col_index,
239
 
                                                 (rev_index + 1,))
240
 
                            
241
 
                            # Recall _line_col_search_order to reset it back to
242
 
                            # the beging.
243
 
                            col_search_order = \
244
 
                                    _line_col_search_order(columns,
245
 
                                                           parent_col_index,
246
 
                                                           col_index)
247
 
                            parent_col_line_index = \
248
 
                                _find_free_column(columns,
249
 
                                                  empty_column,
250
 
                                                  col_search_order,
251
 
                                                  (parent_index - 1,))
252
 
                            _mark_column_as_used(columns,
253
 
                                                 parent_col_line_index,
254
 
                                                 (parent_index - 1,))
255
 
                            lines.append((rev_index,
256
 
                                          parent_index,
257
 
                                          (child_line_col_index,
258
 
                                           parent_col_line_index)))
259
 
                        else :
260
 
                            line_col_index = col_index
261
 
                            if parent_index - rev_index >1:
262
 
                                line_range = range(rev_index + 1, parent_index)
263
 
                                line_col_index = \
264
 
                                    _find_free_column(columns,
265
 
                                                      empty_column,
266
 
                                                      col_search_order,
267
 
                                                      line_range)
268
 
                                _mark_column_as_used(columns,
269
 
                                                     line_col_index,
270
 
                                                     line_range)
271
 
                            lines.append((rev_index,
272
 
                                          parent_index,
273
 
                                          (line_col_index,)))
274
 
        
275
 
        for (child_index, parent_index, line_col_indexes) in lines:
276
 
            (child_col_index, child_color) = linegraph[child_index][1]
277
 
            (parent_col_index, parent_color) = linegraph[parent_index][1]
278
 
            
279
 
            if len(line_col_indexes) == 1:
280
 
                if parent_index - child_index == 1:
281
 
                    linegraph[child_index][2].append(
282
 
                        (child_col_index,
283
 
                         parent_col_index,
284
 
                         parent_color))
285
 
                else:
286
 
                    # line from the child's column to the lines column
287
 
                    linegraph[child_index][2].append(
288
 
                        (child_col_index,
289
 
                         line_col_indexes[0],
290
 
                         parent_color))
291
 
                    # lines down the line's column
292
 
                    for line_part_index in range(child_index+1, parent_index-1):
293
 
                        linegraph[line_part_index][2].append(
294
 
                            (line_col_indexes[0],   
295
 
                             line_col_indexes[0],
296
 
                             parent_color))
297
 
                    # line from the line's column to the parent's column
298
 
                    linegraph[parent_index-1][2].append(
299
 
                        (line_col_indexes[0],
300
 
                         parent_col_index,
301
 
                         parent_color))
302
 
            else:
303
 
                # Broken line
304
 
                # line from the child's column to the lines column
305
 
                linegraph[child_index][2].append(
306
 
                    (child_col_index,
307
 
                     line_col_indexes[0],
308
 
                     parent_color))
309
 
                # Broken line end
310
 
                linegraph[child_index+1][2].append(
311
 
                    (line_col_indexes[0],
312
 
                     None,
313
 
                     parent_color))
314
 
                
315
 
                # Broken line end 
316
 
                linegraph[parent_index-2][2].append(
317
 
                    (None,
318
 
                     line_col_indexes[1],
319
 
                     parent_color))
320
 
                # line from the line's column to the parent's column
321
 
                linegraph[parent_index-1][2].append(
322
 
                    (line_col_indexes[1],
323
 
                     parent_col_index,
324
 
                     parent_color))
325
 
        return (linegraph, revid_index, len(columns))
326
 
    else:
327
 
        return (linegraph, revid_index, 0)
328
 
    
329
 
 
330
 
def _branch_line_col_search_order(columns, parent_col_index):
331
 
    for col_index in range(parent_col_index, len(columns)):
332
 
        yield col_index
333
 
    for col_index in range(parent_col_index-1, -1, -1):
334
 
        yield col_index
335
 
 
336
 
def _line_col_search_order(columns, parent_col_index, child_col_index):
337
 
    if parent_col_index is not None:
338
 
        max_index = max(parent_col_index, child_col_index)
339
 
        min_index = min(parent_col_index, child_col_index)
340
 
        for col_index in range(max_index, min_index -1, -1):
341
 
            yield col_index
342
 
    else:
343
 
        max_index = child_col_index
344
 
        min_index = child_col_index
345
 
        yield child_col_index
346
 
    i = 1
347
 
    while max_index + i < len(columns) or \
348
 
          min_index - i > -1:
349
 
        if max_index + i < len(columns):
350
 
            yield max_index + i
351
 
        if min_index - i > -1:
352
 
            yield min_index - i
353
 
        i += 1
354
 
 
355
 
def _find_free_column(columns, empty_column, col_search_order, line_range):
356
 
    for col_index in col_search_order:
357
 
        column = columns[col_index]
358
 
        has_overlaping_line = False
359
 
        for row_index in line_range:
360
 
            if column[row_index]:
361
 
                has_overlaping_line = True
362
 
                break
363
 
        if not has_overlaping_line:
364
 
            break
365
 
    else:
366
 
        col_index = len(columns)
367
 
        column = list(empty_column)
368
 
        columns.append(column)
369
 
    return col_index
370
 
 
371
 
def _mark_column_as_used(columns, col_index, line_range):
372
 
    column = columns[col_index]
373
 
    for row_index in line_range:
374
 
        column[row_index] = True    
375
 
 
376
 
def same_branch(a, b):
377
 
    """Return whether we think revisions a and b are on the same branch."""
378
 
    if len(a.parent_ids) == 1:
379
 
        # Defacto same branch if only parent
380
 
        return True
381
 
    elif a.committer == b.committer:
382
 
        # Same committer so may as well be
383
 
        return True
384
 
    else:
385
 
        return False
 
1
# -*- coding: UTF-8 -*-
 
2
"""Directed graph production.
 
3
 
 
4
This module contains the code to produce an ordered directed graph of a
 
5
bzr branch, such as we display in the tree view at the top of the bzrk
 
6
window.
 
7
"""
 
8
 
 
9
__copyright__ = "Copyright © 2005 Canonical Ltd."
 
10
__author__    = "Scott James Remnant <scott@ubuntu.com>"
 
11
 
 
12
 
 
13
def linegraph(branch, start, maxnum):
 
14
    """Produce a directed graph of a bzr branch.
 
15
 
 
16
    Returns a list of tuples of (revision, node, lines, parents, children).
 
17
 
 
18
    Node is a tuple of (column, colour) with column being a zero-indexed
 
19
    column number of the graph that this revision represents and colour
 
20
    being a zero-indexed colour (which doesn't specify any actual colour
 
21
    in particular) to draw the node in.
 
22
 
 
23
    Lines is a list of tuples which represent lines you should draw away
 
24
    from the revision, if you also need to draw lines into the revision
 
25
    you should use the lines list from the previous iteration.  Each
 
26
    typle in the list is in the form (start, end, colour) with start and
 
27
    end being zero-indexed column numbers and colour as in node.
 
28
 
 
29
    It's up to you how to actually draw the nodes and lines (straight,
 
30
    curved, kinked, etc.) and to pick the actual colours for each index.
 
31
    """
 
32
    
 
33
    def getdirectparent(childrevid, childindex, childsparents):
 
34
        """Return the revision id of the direct parent
 
35
        
 
36
        The direct parent is the first parent with the same committer"""
 
37
        childrevision = revisions[childindex]
 
38
        directparent = directparentcache[childindex]
 
39
        if directparent is None:
 
40
            for parentrevid in childsparents:
 
41
                if parentrevid in revindex:
 
42
                    parentindex = revindex[parentrevid]
 
43
                    parentrevision = revisions[parentindex]
 
44
                    if childrevision.committer == parentrevision.committer:
 
45
                        # This may be a direct parent, but first check that
 
46
                        # for this parent, there are no other children, who are
 
47
                        # before us in the children list, for which this parent
 
48
                        # is also the direct parent.
 
49
                        # pc in all the below var name stands for parents child
 
50
                        first = True
 
51
                        for pcrevid in revisionchildren[parentindex]:
 
52
                            if pcrevid == childrevid:
 
53
                                break
 
54
                            pcindex = revindex[pcrevid]
 
55
                            pcdirectparent = getdirectparent(pcrevid,
 
56
                                                    pcindex,
 
57
                                                    revisionchildren[pcindex])
 
58
                            if pcdirectparent==parentrevid:
 
59
                                first = False
 
60
                                break
 
61
                        
 
62
                        if first:
 
63
                            directparent = parentrevid
 
64
                            break
 
65
            #no parents have the same commiter
 
66
            if directparent is None:
 
67
                directparent = ""
 
68
            directparentcache[childindex] = directparent
 
69
        return directparent
 
70
    
 
71
    def find_column_from_branchlineid(branchlineid):
 
72
        for index, column in enumerate(columns):
 
73
            if column is not None and column["branchlineid"] == branchlineid:
 
74
                return (index, column)
 
75
        return (None, None)
 
76
    
 
77
    def has_no_nodes_between (column, startindex, endindex):
 
78
        for nodeindex in column["nodeindexes"]:
 
79
            if nodeindex > startindex and nodeindex < endindex:
 
80
                return False
 
81
        return True
 
82
    
 
83
    def append_line (column , startindex, endindex):
 
84
        column["lines"].append((startindex,endindex))
 
85
        if endindex > column["maxindex"] :
 
86
            column["maxindex"] = endindex
 
87
    
 
88
    def append_column (column, minindex):
 
89
        columnindex = None
 
90
        for (i, c) in enumerate(columns):
 
91
            if c is None or (c["branchlineid"] == "finished" \
 
92
                             and c["maxindex"] <= minindex):
 
93
                columnindex = i
 
94
                columns[columnindex] = column
 
95
                break
 
96
        if columnindex is None:
 
97
            columnindex = len(columns)
 
98
            columns.append(column)
 
99
        return columnindex
 
100
    
 
101
    revids = []
 
102
    revindex = {}
 
103
    for (index, revid) in enumerate(reversed( \
 
104
            branch.repository.get_ancestry(start))):
 
105
        if revid is not None:
 
106
            revids.append(revid)
 
107
            revindex[revid] = index
 
108
        if maxnum is not None and index > maxnum:
 
109
            break
 
110
    
 
111
    mainline = branch.revision_history()
 
112
    revisions = branch.repository.get_revisions(revids)
 
113
    revisionparents = branch.repository.get_graph().get_parents(revids)    
 
114
    directparentcache = [None for revision in revisions]
 
115
    
 
116
    # This will hold what we plan to put in each column.
 
117
    # The position of the item in this list indicates the column, and it
 
118
    # it may change if we need to make space for other things.
 
119
    # Each typle in the list is in the form:
 
120
    # (branchlineid, nodeindexes, lines, maxindex)
 
121
    # A item may be None to indicate that there is nothing in the column
 
122
    columns = []
 
123
    
 
124
    linegraph = []
 
125
    
 
126
    lastbranchlineid = 0
 
127
    branchlineids = []
 
128
    revisionchildren = [[] for revision in revisions]
 
129
    notdrawnlines = []
 
130
    
 
131
    for (index, revision) in enumerate(revisions):
 
132
        parents = [parent for parent in revisionparents[index] \
 
133
                   if parent!="null:"]
 
134
        for parentrevid in parents:
 
135
            if parentrevid in revindex:
 
136
                revisionchildren[revindex[parentrevid]]\
 
137
                    .append(revision.revision_id)
 
138
        
 
139
        children = revisionchildren[index]
 
140
        
 
141
        #We use childrevid, childindex, childbranchlineid often, so cache it
 
142
        children_ext = []
 
143
        for childrevid in children:
 
144
            childindex = revindex[childrevid]
 
145
            children_ext.append((childrevid,
 
146
                                 childindex,
 
147
                                 branchlineids[childindex]))
 
148
        
 
149
        linegraph.append([revision, None,
 
150
                          [], parents, children])
 
151
        
 
152
        branchlineid = None
 
153
        
 
154
        if revision.revision_id in mainline:
 
155
            branchlineid = 0
 
156
        else:
 
157
            #Try and see if we are the same branchline as one of our children
 
158
            #If we are, use the same branchlineid
 
159
            for (childrevid, childindex, childbranchlineid) in children_ext: 
 
160
                childsparents = revisionparents[childindex]
 
161
                
 
162
                if len(children) == 1 and len(childsparents) == 1: 
 
163
                    # one-one relationship between parent and child
 
164
                    branchlineid = childbranchlineid
 
165
                    break
 
166
                
 
167
                #Is the current revision the direct parent of the child?
 
168
                if childbranchlineid != 0 and revision.revision_id == \
 
169
                        getdirectparent(childrevid, childindex, childsparents):
 
170
                    branchlineid = childbranchlineid
 
171
                    break
 
172
        
 
173
        if branchlineid is None:
 
174
            branchlineid = lastbranchlineid = lastbranchlineid + 1
 
175
        
 
176
        branchlineids.append(branchlineid)
 
177
        
 
178
        (columnindex, column) = find_column_from_branchlineid(branchlineid)
 
179
        
 
180
        if columnindex is None:
 
181
            for (childrevid, childindex, childbranchlineid) in children_ext:
 
182
                (i, c) = find_column_from_branchlineid(childbranchlineid)
 
183
                if c is not None and c["maxindex"] <= index:
 
184
                    (columnindex, column) = (i, c)
 
185
                    break
 
186
    
 
187
 
 
188
        
 
189
        if columnindex is None:
 
190
            minindex = index
 
191
            for childrevid in children:
 
192
                childindex = revindex[childrevid]
 
193
                if childindex<minindex:
 
194
                    minindex = childindex
 
195
            
 
196
            column = {"branchlineid": branchlineid,
 
197
                      "nodeindexes": [index],
 
198
                      "lines": [],
 
199
                      "maxindex": index}
 
200
            columnindex = append_column(column, minindex)
 
201
        else:
 
202
            column["branchlineid"] = branchlineid
 
203
            column["nodeindexes"].append(index)
 
204
        
 
205
        opentillparent = getdirectparent(revision.revision_id, index, parents)
 
206
        if opentillparent == "":
 
207
            if len(parents)>0:
 
208
                opentillparent = parents[0]
 
209
            else:
 
210
                opentillparent = None
 
211
        
 
212
        if opentillparent is not None and opentillparent in revindex:
 
213
            parentindex = revindex[opentillparent]
 
214
            if parentindex > column["maxindex"]:
 
215
                column["maxindex"] = parentindex
 
216
        
 
217
        for (childrevid, childindex, childbranchlineid) in children_ext: 
 
218
            (childcolumnindex, childcolumn) = \
 
219
                find_column_from_branchlineid(childbranchlineid)
 
220
            
 
221
            if index-childindex == 1 or childcolumnindex is None:
 
222
                append_line(column,childindex,index)
 
223
            elif childcolumnindex >= columnindex and \
 
224
                    has_no_nodes_between(childcolumn, childindex, index):
 
225
                append_line(childcolumn,childindex,index)
 
226
            elif childcolumnindex > columnindex and \
 
227
                    has_no_nodes_between(column, childindex, index):
 
228
                append_line(column,childindex,index)
 
229
            elif childcolumnindex < columnindex and \
 
230
                    has_no_nodes_between(column, childindex, index):
 
231
                append_line(column,childindex,index)
 
232
            elif childcolumnindex < columnindex and \
 
233
                    has_no_nodes_between(childcolumn, childindex, index):
 
234
                append_line(childcolumn,childindex,index)
 
235
            else:
 
236
                append_column({"branchlineid": "line",
 
237
                                "nodeindexes": [],
 
238
                                "lines": [(childindex,index)],
 
239
                                "maxindex": index}, childindex)
 
240
        
 
241
        for (columnindex, column) in enumerate(columns):
 
242
            if column is not None and column["maxindex"] <= index \
 
243
                    and column["branchlineid"] != "finished":
 
244
                for nodeindex in column["nodeindexes"]:
 
245
                    linegraph[nodeindex][1] = (columnindex,
 
246
                                               branchlineids[nodeindex])
 
247
            
 
248
                for (childindex, parentindex) in column["lines"]:
 
249
                    notdrawnlines.append((columnindex, childindex, parentindex))
 
250
                
 
251
                column["branchlineid"] = "finished"
 
252
                column["nodeindexes"] = []
 
253
                column["lines"] = []
 
254
        
 
255
        for (lineindex,(columnindex, childindex, parentindex))\
 
256
                in enumerate(notdrawnlines):
 
257
            
 
258
            childnode = linegraph[childindex][1]
 
259
            parentnode = linegraph[parentindex][1]
 
260
            if childnode is not None and parentnode is not None:
 
261
                notdrawnlines[lineindex] = None
 
262
                if parentindex>childindex+1:
 
263
                    #out from the child to line
 
264
                    linegraph[childindex][2].append(
 
265
                        (childnode[0],    #the column of the child
 
266
                         columnindex,     #the column of the line
 
267
                         parentnode[1]))
 
268
                    
 
269
                    #down the line
 
270
                    for linepartindex in range(childindex+1, parentindex-1):
 
271
                        linegraph[linepartindex][2].append(
 
272
                            (columnindex, #the column of the line
 
273
                             columnindex, #the column of the line
 
274
                             parentnode[1]))
 
275
                    
 
276
                    #in to the parent
 
277
                    linegraph[parentindex-1][2].append(
 
278
                        (columnindex,     #the column of the line
 
279
                         parentnode[0],   #the column of the parent
 
280
                         parentnode[1]))
 
281
                else:
 
282
                    #child to parent
 
283
                    linegraph[childindex][2].append(
 
284
                        (childnode[0],    #the column of the child
 
285
                         parentnode[0],   #the column of the parent
 
286
                         parentnode[1]))
 
287
        
 
288
        notdrawnlines = [line for line in notdrawnlines if line is not None]
 
289
        
 
290
 
 
291
    return (linegraph, revindex, revisions)
 
292
 
 
293
 
 
294
def same_branch(a, b):
 
295
    """Return whether we think revisions a and b are on the same branch."""
 
296
    if len(a.parent_ids) == 1:
 
297
        # Defacto same branch if only parent
 
298
        return True
 
299
    elif a.committer == b.committer:
 
300
        # Same committer so may as well be
 
301
        return True
 
302
    else:
 
303
        return False