Hi All
I have made all the changes.
Thanks & Regards
--Avijit Basak
On Sat, 25 Sept 2021 at 17:14, Gilles Sadowski <gillese...@gmail.com> wrote:
> Hello.
>
> Le sam. 25 sept. 2021 à 08:06, Avijit Basak <avijit.ba...@gmail.com> a
> écrit :
> >
> > Hi All
> >
> > I have created a PR (#197) to merge my changes for Genetic
> Algorithm. Although the build has passed locally for my components
> (commons-math4-genetics and example-genetics) the PR build failed with unit
> test case errors for legacy(commons-math4-legacy) modules.
>
> Please note that maven modules should named "commons-math-Xxx";
> as Alex mentioned previously, you should remove the spurious "4" in
> the "genetics" module (and I'd personally favour "ga" over "genetics", as
> per a previous discussion). Thus:
> commons-math-ga
> with top-level package
> org.apache.commons.math4.ga
>
> > I have not changed anything in the legacy module. So not sure what is
> causing those issues.
>
> It is caused by a randomized test ("SimplexOptimizerTest", see the log[1]).
>
> > Could anyone kindly guide me how to pass the PR build in this scenario.
>
> You might try to resubmit the PR.
> [Anyways the build will be performed after you commit the changes
> indicated above.]
>
> Best regards,
> Gilles
>
> > The local build logs are attached herewith for reference.
> >
> > Local build command: mvn clean verify apache-rat:check checkstyle:check
> pmd:check spotbugs:check javadoc:javadoc
> > PR Link: https://github.com/apache/commons-math/pull/197
> >
> > Thanks & Regards
> > -- Avijit Basak
> >
>
> [1] https://app.travis-ci.com/github/apache/commons-math/builds/238471866
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>
--
Avijit Basak
