[Monotone-devel] Re: ViewMTN 0.01

[Bruce Stephens]

address@hidden (Grahame Bowland) writes:

[...]

> If a revision is signed with multiple branches it should end up
> showing multiple branches at once? I'm not quite sure what it does
> in this case!

I presume it'll end up in multiple branches.  I was assuming that
propagates would produce a revision somewhat like merges, so the
structure would show the revision having two ancestors.  But it
doesn't: if you want to know, you have to interpret the changelog.  So
that's simpler than I thought.

>> I think it would be nice to show arrowheads (because this is a
>> directed graph), and maybe show merges differently (as monotone-viz
>> does).  However, those two are really minor, IMHO.
>
> I've changed it to be a directed graph, and the revision we're
> determining the ancestry of is now shown in blue. I'll look into
> showing branches differently, thanks for the suggestion :-)

Ah, but now the graph is up the other way, with the heads at the
bottom.  That's consistent with the way monotone-viz does it, which is
good.  But it means that I (often) have to scroll to see the head
which the changelog refers to.

I'm not sure what the right thing to do is.  It's simple to reverse
things, if that seems better (but I'm not sure if it is):

# 
# patch "monotone.py"
#  from [428483fe538c6687671cdd2dd64841f217037a25]
#    to [f87f406a1cc397adeddb23c04613efdc1c9542ae]
# 
--- monotone.py
+++ monotone.py
@@ -149,6 +149,7 @@
                        rv['cached'] = True
                        return rv
                contents = "digraph ancestry {"
+               contents += "edge [dir=back];\n"
                revisions = sets.Set()
                for attrs in self.ancestry(id, limit):
                        if not attrs.has_key("Revision") or not 
attrs.has_key("Ancestor"):
@@ -158,7 +159,7 @@
                        for ancestor in attrs['Ancestor']:
                                if len(ancestor) == 0: continue
                                revisions.add(ancestor)
-                               contents += '"%s"->"%s"\n' % (ancestor, 
revision)
+                               contents += '"%s"->"%s"\n' % (revision, 
ancestor)
                for revision in revisions:
                        opts = 'fontsize=8,shape=box,href="revision.psp?id=%s"' 
% (urllib.quote(revision))
                        if revision == id: opts += ",color=blue"