[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Numbers] More modules (Was: [Statistics] Port codes from Commons Math)

Hello Gimhana.

On Thu, 17 May 2018 23:32:15 +0200, Gilles wrote:
Hi Gimhana.

On Fri, 18 May 2018 00:16:04 +0530, Gimhana Nadeeshan wrote:
Hi all,

We might want to create a public branch for that work in order to
merge PRs more quickly without risk of breaking "master".
What do you think?  Eric?

I ported the Statistics Interval Module and would like to get your reviews.
How should I make the Pull request ?

I've had a quick look, but as I've noted in the JIRA comment, revamping
stat codes from "Commons Math" into "Commons Statistics" is a lot of
work, part redesign, and part porting/adaptation.

You could more easily get acquainted with the formalities of contributing
to "Commons" (e.g. conventions, reports, doc, etc.) by tackling tasks
where there is no redesign involved (except the simplification concerning
e.g. the handling of exceptions).

Would you be interested in examining whether the codes which I mentioned in [1] could be ported to "Commons Numbers"? These are smaller tasks to
which I could devote some review time, while for "Commons Statistics" I
don't want to enter an open-ended discussion on how to improve the design
without the involvment of users of the "Commons Math" stat codes.



I've just created a new branch on the repository; please make
all PR refer to "task_STATISTICS-5".
I also suggest that you create finer-grained "sub-tasks" of


Best Regards,

On 5 May 2018 at 18:50, Gilles <gilles@xxxxxxxxxxxxxxxxxxxxx> wrote:

Hi Gimhana.

On Sat, 5 May 2018 15:50:43 +0530, Gimhana Nadeeshan wrote:

Hello all,

As I proposed early I would like to begin port code from Commons-math
<> to Commons-statistics
(For further details refer my  GSoC Proposal

though I'm not selected this year)

This is my proposed architecture in brief

1. Commons-Statistics-Core => Frequency and StatUtils classes (Can add
   more common classes while implementing)
   2. Commons-Statistics-Correlation
   3. Commons-Statistics-Descriptive
   4. Commons-Statistics-Inference
   5. Commons-Statistics-Interval
   6. Commons-Statistics-Ranking
   7. Commons-Statistics-Regression

Nit-pick: module names have no capital in them (just a convention).
So: "commons-statistics-core" rather than "Commons-Statistics-Core", etc.

While I referring Commons-Geometry

No need to refer to that project since "Commons Statistics" has been
set up:

The code repository is here:
It already contains a "commons-statistics-distribution" module whose layout can be duplicated in the modules which you are proposing above
(with appropriate changes of course).

ported code to get a head start , I
found that each module inside, contain a pox.xml file. Are they
as separate projects and then group in the same package? I'm asking
Since I'm new to code porting :-).

A requirement is that no package should be shared between different
modules; by convention, the top-level package of module
would be

[And so on for the other modules. But I'd suggest you start with one.]

If so in here should I create all 7 projects and then group those in same

No, the project is "Commons Statisitics" and it would contain several _maven_ modules, each of which should ultimately map to a _JPMS_ (JDK9)

Firstly I suppose to start port Ranking Module as it has less
dependencies comparing to others.

Fine. But don't forget to browse through the JIRA issues of Commons
Math (CM) for things that would need fixing. Whenever it's the case,
please open a report in the new JIRA project (linking to the CM
report), and post here your proposed solution (or questions).

We might want to create a public branch for that work in order to
merge PRs more quickly without risk of breaking "master".
What do you think?  Eric?

Would someone help me to get a head start ??

What else do you need?

Best regards,

Best Regards,


To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxxxxxx

( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-commons-developers/msg06337.html on line 279
Call Stack
10.0008372888{main}( ).../msg06337.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-commons-developers/msg06337.html on line 279
Call Stack
10.0008372888{main}( ).../msg06337.html:0