#12141: Implement finite algebras
-------------------------------------+-------------------------------------
Reporter: johanbosman | Owner: AlexGhitza
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.2
Component: algebra | Resolution:
Keywords: sd51 | Merged in:
Authors: Johan Bosman, | Reviewers: Travis Scrimshaw
Peter Bruin, Michiel Kosters | Work issues:
Report Upstream: N/A | Commit:
Branch: | 6dd9dd6c0c36a474642abc3952b46f98a5ceaad5
public/algebras/finite_algebras-12141| Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by pbruin):
Replying to [comment:28 tscrim]:
> I've changed the inverses to use the `@lazy_attribute` and so
`y._inverse` is automatically set when calling `x._inverse`. Everything is
now `EXAMPLES::` and "finite-dimensional".
Actually I had forgotten that the inverse was already cached in the first
place thanks to `_inverse`... Anyway, I do like your idea of using
`@lazy_attribute` for this.
As for `EXAMPLE[S]` and `TEST[S]`, it is true that the documentation only
mentions the plural and this is used predominantly in the Sage library,
but I don't see any reason for preferring the plural regardless of the
actual number of examples/tests. I don't feel too strongly about it
either, but in general I wouldn't recommend to a reviewer to make such
changes when it comes to matters of taste. (Although this only applies
here if `EXAMPLE[S]` is indeed a matter of taste.)
I will try to do a final check soon, unless someone else does it first.
--
Ticket URL: <http://trac.sagemath.org/ticket/12141#comment:29>
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.
