Re: [Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-28 Thread Gilles Sadowski
Hi. > Seeing the code in this package i agree that some more exceptions should be > created so as to cover cases such as the following: > > >Package: > org.apache.commons.math.analysis.integration.UnivarieteRealIntegratorImpl > > >public double getResult() throws IllegalStateException { > >

Re: [Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-28 Thread Michael Giannakopoulos
Also, the file 'ResizableDoubleArray.java' in the package 'org.apache.commons.math.util' contains the following code: >if (contraction < expansion) { >throw MathRuntimeException.createIllegalArgumentException( > LocalizedFormats.CONTRACTION_CRITERIA_SMALLER_THAN_EXPANSION_FACTOR, >

Re: [Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-28 Thread Michael Giannakopoulos
Hello to everyone, Seeing the code in this package i agree that some more exceptions should be created so as to cover cases such as the following: >Package: org.apache.commons.math.analysis.integration.UnivarieteRealIntegratorImpl >public double getResult() throws IllegalStateException { >

Re: [Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-27 Thread Phil Steitz
On Thu, Jan 27, 2011 at 6:50 AM, Gilles Sadowski wrote: > Hi. > >> > I need some clarifications before embarking into the clean-up of that >> > package. >> > >> > The "integrate" method of interface "UnivariateRealIntegrator" is defined >> > as: >> >  double integrate(UnivariateRealFunction f, dou

Re: [Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-27 Thread Gilles Sadowski
Hi. > > I need some clarifications before embarking into the clean-up of that > > package. > > > > The "integrate" method of interface "UnivariateRealIntegrator" is defined > > as: > >  double integrate(UnivariateRealFunction f, double min, double max) > >        throws ConvergenceException, MathU

Re: [Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-26 Thread Phil Steitz
On Wed, Jan 26, 2011 at 7:23 AM, Gilles Sadowski wrote: > Hi. > > I need some clarifications before embarking into the clean-up of that > package. > > The "integrate" method of interface "UnivariateRealIntegrator" is defined > as: >  double integrate(UnivariateRealFunction f, double min, double ma

[Math] Refactoring of package "analysis.integration" (MATH-501)

2011-01-26 Thread Gilles Sadowski
Hi. I need some clarifications before embarking into the clean-up of that package. The "integrate" method of interface "UnivariateRealIntegrator" is defined as: double integrate(UnivariateRealFunction f, double min, double max) throws ConvergenceException, MathUserException, IllegalArg