#18720: Change diagram algebra basis set partitions from list to generator
-------------------------------------+-------------------------------------
Reporter: ghseeli | Owner:
Type: enhancement | Status: new
Priority: minor | Milestone: sage-6.8
Component: algebra | Resolution:
Keywords: days65, partition | Merged in:
algebra, diagram algebra | Reviewers:
Authors: ghseeli | Work issues:
Report Upstream: N/A | Commit:
Branch: | 879e2b6d89475802810e573bf8d396e401ab4d06
u/ghseeli/change_diagram_algebra_basis_set_partitions_from_list_to_generator|
Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by tscrim):
Replying to [comment:11 ghseeli]:
> I am sorry, I don't quite get what "make the basis indexing set iterate
over elements of itself" means because I don't quite understand the
structure of the `LazyFamily` in the context of `CombinatorialFreeModule`
yet. My concern is: should users expect that calling for the support of an
element return a uniform type? I will commit what I have so you can take a
look.
You are correct, it should be a uniform type. What the `LazyFamily` does
in `CombinatorialFreeModule.basis()` (in this case) is iterate over the
`AbstractPartitionDiagrams` class and then makes that into an element of
the CFM, but only when required. However from looking at it, it should be
working properly.
Also I think `range(-len(tst)/2,len(tst)/2+1)` might give you trouble
because it's doing python int division (where `1/2 == 0`). And `except
TypeError, ValueError:` should be `except (TypeError, ValueError):`.
--
Ticket URL: <http://trac.sagemath.org/ticket/18720#comment:13>
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/d/optout.
