-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce Policy Methods for MOEA/D-DE #293
Conversation
=> PBI => Tchebycheff => Weighted
=> BayesianBootStrap : Working => Uniform : **NOT WORKING**
include/ensmallen_bits/moead/decomposition_policies/pbi_decomposition.hpp
Show resolved
Hide resolved
const VecType& /* idealPoint */, | ||
const VecType& candidateFitness) | ||
{ | ||
return arma::dot(weight, candidateFitness); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That one is so easy, love it.
include/ensmallen_bits/moead/weight_init_policies/uniform_init.hpp
Outdated
Show resolved
Hide resolved
include/ensmallen_bits/moead/weight_init_policies/uniform_init.hpp
Outdated
Show resolved
Hide resolved
auto BinomialCoefficient = | ||
[](size_t n, size_t k) -> double | ||
{ | ||
return std::tgamma(n + 1) / (std::tgamma(k + 1) * std::tgamma(n - k + 1)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, this is what @coatless suggested?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this is a binomial coefficient. That one was Dirichilet
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
include/ensmallen_bits/moead/decomposition_policies/pbi_decomposition.hpp
Outdated
Show resolved
Hide resolved
include/ensmallen_bits/moead/decomposition_policies/tchebycheff_decomposition.hpp
Outdated
Show resolved
Hide resolved
include/ensmallen_bits/moead/decomposition_policies/weighted_decomposition.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks already great to me, I guess once the comments are added, this is pretty much ready.
…osition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
…ecomposition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
…f_decomposition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Jeez, sorry for making you do all that minor fixes. |
No problem, the rest looks great to me, I guess if we can fix the windows issue, we are all set. |
…osition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
…f_decomposition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
…ecomposition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
…osition.hpp Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
Co-authored-by: Marcus Edel <marcus.edel@fu-berlin.de>
@zoq I think this one's good to go. |
Will approve once appveyor comes back green. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No further comments from my side, looks great.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Second approval provided automatically after 24 hours. 👍
Thanks, for another great contribution. |
Continuation of #269
Weight decomposition
Weigh Init
Testing
@zoq @coatless @say4n