#13442: provide gcd for new polynomial rings through _gcd_univariate_polynomial
-------------------------------------+-------------------------------------
Reporter: saraedum | Owner: AlexGhitza
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.4
Component: basic arithmetic | Resolution:
Keywords: sd59 | Merged in:
Authors: Julian Rueth | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/saraedum/ticket/13442 | 8126ef15551e8d8ed5d0edb2d559f5a9fd87bcf1
Dependencies: #13441 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by saraedum):
Replying to [comment:15 bruno]:
> I agree with [#comment:5 pbruin] on `self` versus `this polynomial`. I
find the documentation clearer with `self`, and this seems to be more
consistent with the rest of Sage.
Both approaches have their drawbacks. `self` makes it harder to read if
you do not know about python, i.e., if a 'user' consults the help. 'this
polynomial' makes things slightly more difficult to understand if you know
about `self`.
I have been asked on different tickets to replace `self` with something
more appropriate. Sage is not really consistent with this.
> Tell me what do you think of this, I'll positive review the ticket then.
All tests passed with the current commits.
I do not really care how we do this in sage. Either way is fine with me.
--
Ticket URL: <http://trac.sagemath.org/ticket/13442#comment:16>
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.
