#17674: xgcd is badly named on non PID domains
---------------------------------+------------------------
Reporter: vdelecroix | Owner:
Type: PLEASE CHANGE | Status: new
Priority: major | Milestone: sage-6.5
Component: PLEASE CHANGE | Resolution:
Keywords: | Merged in:
Authors: | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
Dependencies: | Stopgaps:
---------------------------------+------------------------
Comment (by bruno):
Replying to [comment:3 jdemeyer]:
> Replying to [comment:2 bruno]:
> > * My preferred solution is the one chosen by Flint and NTL, keeping
the name `xgcd`;
> FLINT ''does not'' use the name `xgcd`, it uses the name
`fmpz_poly_xgcd`. This may look like nitpicking, but it's really not. The
FLINT call is explicit that it's for `fmpz_poly`: polynomials over `ZZ`.
When you use the generic `xgcd` name to mean both the "real" xgcd and the
"fake" xgcd-times-resultant depending on the input, that's confusing.
In Sage you have the object-oriented syntax `object.method()` while in
Flint there is some kind of naming convention `type_function()`. And you
have `fmpz_poly_xgcd` exactly in the same way as you have `fmpz_xgcd` or
`fmpq_poly_xgcd`, so one could argue, in the same way as you argue for the
`xgcd` function in Sage, that a user expects these functions to implement
the same mathematical function in the three cases. I am not sure (even if
you think so) that the situation in Sage is much more confusing than the
situation in Flint. Actually, I do not think it is really confusing in any
case.
--
Ticket URL: <http://trac.sagemath.org/ticket/17674#comment:4>
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.
