#13993: Implementation of plotting cover labels for posets
-----------------------------------+----------------------------------------
Reporter: stumpc5 | Owner: sage-combinat
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.10
Component: combinatorics | Resolution:
Keywords: poset | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Christian Stump | Merged in:
Dependencies: | Stopgaps:
-----------------------------------+----------------------------------------
Changes (by ncohen):
* status: needs_review => needs_work
Comment:
Looks like this patch should be rebased !
Otherwise :
{{{
[ (v,w,l) for (v,w),l in cover_labels.iteritems() ]
}}}
Looks more efficient than
{{{
[ (v,w,cover_labels[(v,w)]) for (v,w) in cover_labels ]
}}}
But what you want to do, I think, is this :
{{{
for u,v in graph.edges(labels = False):
graph.set_edge_label(cover_labels.get((u,v),''))
}}}
And that also means that you do not need to use a boolean cover_labels
variable anymore.
Yes. I know. Who the hell cares ?
It's just that it hurts to see computers do more things that they need to.
But of course it will not make the slightest difference in speed.
Of course `:-P`
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13993#comment:3>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
