-
Notifications
You must be signed in to change notification settings - Fork 44
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
Fix pareto dominance definition #174
Fix pareto dominance definition #174
Conversation
…ess values are same along one axis
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.
Hi @hannah-jan
Thank you very much for the PR!
There seems to be a test that does not pass because the variable mask
seems undefined?
Apart from that it looks good to me! 😃
…nd compute_masked_pareto_dominance
Hello!! @Lookatator thanks for the comment!! Sorry I did it in a rush (classic me) so put it in the wrong definition. Both Pareto Dominance definitions should be updated now |
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #174 +/- ##
===========================================
- Coverage 92.31% 90.96% -1.36%
===========================================
Files 103 125 +22
Lines 5816 7657 +1841
===========================================
+ Hits 5369 6965 +1596
- Misses 447 692 +245 ☔ View full report in Codecov by Sentry. |
30ea208
into
adaptive-intelligent-robotics:develop
* feat(algo): add DCG-MAP-Elites whose most recent version is called DCRL-ME (#167) * fix: Fix pareto dominance definition to account for solutions which have the same fitness values along one axis (#174) * docs(contribution): clarify contribution process (#171) * feat: Upgrade Library Versions and Python Version (#187) --------- Co-authored-by: Maxence Faldor <m.faldor22@imperial.ac.uk> Co-authored-by: Hannah Janmohamed <hannah.janmohamed21@imperial.ac.uk> Co-authored-by: Felix Chalumeau <f.chalumeau@instadeep.com>
This PR:
pareto_front.py
to account for solutions which have the same fitness values along one axis.