#12630: Add representations of quivers and quiver algebras to sage
------------------------------------------------------------------------------+
Reporter: JStarx |
Owner: AlexGhitza
Type: enhancement |
Status: needs_work
Priority: major |
Milestone: sage-5.11
Component: algebra |
Resolution:
Keywords: algebra, quiver, module, days49 |
Work issues: Use existing infrastructure for Homspase and Morphism.
Report Upstream: N/A |
Reviewers:
Authors: Jim Stark, Simon King, Mathieu Guay-Paquet, Aladin Virmaux |
Merged in:
Dependencies: #12412, #12413 |
Stopgaps:
------------------------------------------------------------------------------+
Comment (by SimonKing):
Replying to [comment:84 virmaux]:
> Here is the modified piece of code:
>
> {{{
> def __init__(self, data=None, pos=None, loops=True, format=None,
> boundary=(), weighted=True, implementation='c_graph',
> sparse=True, vertex_labels=True, name=None,
> multiedges=True,
convert_empty_dict_labels_to_None=False, **kwds):
> }}}
OK. This makes sense.
In order to not duplicate work, I propose that I "refactor the factory"
now and directly incorporate the change (`boundary=()`) and the fix for
the only failing test.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12630#comment:85>
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.
For more options, visit https://groups.google.com/groups/opt_out.