Closed yurilavinas closed 6 years ago
Missing test
Thanks for the contribution! Will take a look soon (probably over the weekend).
In case with work nees review, please let me know!
Hi Yuri, thanks for your contribution. Could you please:
fully document proposed function neighborhood_cga()
(see any function for reference - e.g., scalarization_wt()
break your PR into smaller, specific ones, so that they can be reviewed quicker (see https://blog.viacom.tech/2017/08/23/the-art-of-pull-requests/ for reference). I would suggest that this big PR be divided into at least 2 (one for the summary, another for the CGA). Probably 3 would be even better - one for the summary, one for modularizing the neighborhoods (like it is done for the variation_, scalarization_ etc.), and a third for adding the CGA neighborhood option.
Thanks a lot, Felipe
Just for reference, please take a look at the Writing extensions for MOEADr vignette.
Summary changes:
Latin Hypercube Sample (LHS):