#20280: little cleanup of hexad.py
-------------------------+-------------------------------------------------
Reporter: | Owner:
chapoton | Status: needs_review
Type: | Milestone: sage-7.2
enhancement | Resolution:
Priority: minor | Merged in:
Component: game | Reviewers:
theory | Work issues:
Keywords: | Commit:
Authors: | e50d9c63315f3641ba3f184b8b82679fa11a9b56
Frédéric Chapoton | Stopgaps:
Report Upstream: N/A |
Branch: |
public/20280 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by tscrim):
A few comments:
- Can you put a line break in the `See the docstrings...` sentence?
- Why does `Minimog` need to inherit from `SageObject`? Can't it just
inherit from `object` to be a new-style (Python) class?
- Should we (I am willing to do these too) also do the following:
* Make the error message start with a lowercase letter and not end in a
period.
* Put some of the module level doc into latex/code formatting.
* Cleanup the code of `print_kitten`.
--
Ticket URL: <http://trac.sagemath.org/ticket/20280#comment:5>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
