#5422: [with patch, needs work] Quadratic forms polynomial
-----------------------------+----------------------------------------------
Reporter: aly.deines | Owner: justin
Type: enhancement | Status: new
Priority: trivial | Milestone: sage-3.4.2
Component: quadratic forms | Keywords: quadratic forms
-----------------------------+----------------------------------------------
Comment(by was):
REFEREE REPORT:
* change it to
{{{
EXAMPLES::
sage: stuff
}}}
for the new ReST format.
* Don't hardcode only x, i.e., change this:
{{{
polynomial(self):
}}}
to
{{{
polynomial(self, names='x'):
}}}
Yes, "names" sounds funny, but is traditional in sage for this.
* Change "Input:" and "Output:" to be all caps and on their own line,
like in the rest of sage.
* This is silly code {{{[R.gens()[i] for i in range(n)]}}} because
{{{R.gens()}}} is almost the same thing. If you really need a list do
{{{list(R.gens())}}}.
* This worries me: {{{(M*V).dot_product(V) }}}. Should it be
{{{(V*M).dot_product(V)}}}?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5422#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
