#20243: Count real roots of polynomials using Sturm sequences
-------------------------------------+-------------------------------------
Reporter: kedlaya | Owner: kedlaya
Type: enhancement | Status: needs_work
Priority: minor | Milestone: sage-7.2
Component: algebra | Resolution:
Keywords: polynomials, | Merged in:
root-counting, days71 | Reviewers:
Authors: Kiran Kedlaya | Work issues:
Report Upstream: N/A | Commit:
Branch: | 716f4650f5288e639823caa5ec07da14751d094c
u/kedlaya/count_real_roots_of_polynomials_using_sturm_sequences| Stopgaps:
Dependencies: #20256 |
-------------------------------------+-------------------------------------
Changes (by kdilks):
* status: needs_review => needs_work
Comment:
Patchbot doesn't like something about the way errors are raised. I think
it would be a bit more straightforward to make the arguments be
{{{(self,lower_bound=None,upper_bound=None)}}}, and just not allow for the
option of specifying the bounds as a two-element list.
In terms of the names of functions, I think {{{count_roots_in_interval}}}
would make more sense as {{{number_of_roots_in_interval}}} (that's the
naming convention used for most combinatorial statistics). And
{{{has_all_real_roots}}} should probably be named {{{is_real_rooted}}}.
Things that return a boolean almost always start with {{{is_}}}, and
calling polynomials with all real roots "real rooted" is prevalent, at
least in the literature that I read.
--
Ticket URL: <http://trac.sagemath.org/ticket/20243#comment:37>
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.
