#8547: implement hidden markov models in Cython from scratch (so can remove the
GHMM standard package from Sage)
---------------------------+------------------------------------------------
Reporter: was | Owner: amhou
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.4
Component: statistics | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
---------------------------+------------------------------------------------
Comment(by jason):
Replying to [comment:13 was]:
> * Jason said above that IntList.sum doesn't have a doctest for the
overflow case... but it does, so I don't know what he meant.
I meant that the doctest looks like this:
{{{
Note that there can be overflow, since the entries are C ints::
sage: a = stats.IntList([2^30,2^30]); a
[1073741824, 1073741824]
}}}
That's it. There's no test there; you're just creating a list, not
summing anything.
>
> * I read the source code for {{{sage.misc.misc_c.normalize_index}}} and
cannot bring myself to use that in this situation. That function
actually returns a Python *list* of Python ints for every single index
into the list that is being sliced! That would easily lead to factor of
50-100 slowdowns on realistic operations:
> This stats code I'm writing is really meant to be industrial strength --
the sort of code maybe somebody would use in "realtime" processing of
large datastreams. I don't want slow functions anywhere in there.
I agree.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8547#comment:14>
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 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.
