#16644: LinearCode.cardinality category breakage
-------------------------------------+-------------------------------------
Reporter: ppurka | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: coding theory | Resolution:
Keywords: | Merged in:
Authors: Travis Scrimshaw, | Reviewers: Punarbasu
Punarbasu Purkayastha | Purkayastha, Travis Scrimshaw
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/categories/new_parent_linear_codes-16644|
709878e72911d2630a021ed0485c4d7a099abaff
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Changes (by tscrim):
* status: needs_work => needs_review
Comment:
@vbraun I would argue that `AttributeError` is the correct error: the
attribute should be there according to the category, but it is not because
it is an old-style parent. I'd rather spend my time fixing the problem and
educating people about categories rather than trying to come up with a
different error message. This solves the issue in the ticket description
and we shouldn't hold up this ticket -- the error message should be on a
separate ticket.
@ncohen You actually were right in comment:4. I wonder if you're starting
to understand categories. :P
@ppurka I didn't look at/see the `__len__` method. I've changed that into
`cardinality` and made `__len__` into an alias (so it will appear in the
documentation).
--
Ticket URL: <http://trac.sagemath.org/ticket/16644#comment:24>
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.
