#11572: Add new probabillity dsitributions
---------------------------+------------------------------------------------
Reporter: Kamhamea | Owner: amhou
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.7.2
Component: statistics | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author: Mato Nagel
Merged: | Dependencies:
---------------------------+------------------------------------------------
Comment(by Kamhamea):
> you seem to have subsumed the patch at #9080 in this patch. You should
instead apply that patch to your Sage, and then create your new patch on
top of that one.
Well, that's exactly what I did. Only I transferred the changes of that
patch into my patch by my merge software line by line. It is not the way
it is meant to be done, is it? I'm quite unfamiliar with mercurial version
control system. So to say I'm unable to rewind that patch. Of course I
didn't meant to spoil the credits of patch #9080 developer. As you can see
I didn't change his name in the header.
> A very minor comment - you should set up your HG defaults
Maybe I can do that :)
> Also, a point of information related to your email to sage-devel;
although you add tests for the new distributions, that wouldn't remove the
need for #11514. It would be a good step in that direction, though!
To be honest I don't know exactly what is meant by that. Please give some
example. but you'd better do that at #11514.
> More substantively, I think (my opinion only) that based on the
[http://groups.google.com/group/sage-
devel/browse_thread/thread/8febda1b92330c85/9cea7249b186d8e5 discussion
about this on sage-devel], the binomial and other "discrete" distributions
should somehow be separated, if only for user convenience.
This needs rethinking. Two points to mention:
1. From a programmers perspective, it is too much redundancy. The
implementation of discrete and continuous distribution is almost the same
except plotting. Well we can create a base class that implements all
functionality except plotting and help. Still I believe its superfluous
work.
2. From a user perspective, especially a newcommer's, it is not likely
that he/she knows, for instance, that binomial distribution is a discrete
one and that an other class has to be used. So we may create frustration
at this end too. Besides R doesn't make the distinction either
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11572#comment:3>
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.
