fixed cstrs_self_adpative integer problem handling #465

esa / pagmo2

A C++ platform to perform parallel computations of optimisation tasks (global and local) via the asynchronous generalized island model.

https://esa.github.io/pagmo2/

GNU General Public License v3.0

808 stars 160 forks source link

Closed SchwarzschildX closed 2 years ago

SchwarzschildX commented 3 years ago

made the changes and tested them. works.

darioizzo commented 3 years ago

Can you add a unit test for this? In https://github.com/esa/pagmo2/blob/master/tests/cstrs_self_adaptive.cpp ? The test should fail in the old version and pass in the new version and should test all newly added lines.

SchwarzschildX commented 2 years ago

Sorry for the long delay. Just added a test to check the correct forwarding of the integer dimension from the udp to cstrs_self_adaptive.

bluescarni commented 2 years ago

@SchwarzschildX can you merge master into your branch?

SchwarzschildX commented 2 years ago

@bluescarni Of course! Hope I did it right ;)

bluescarni commented 2 years ago

Thanks!