-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add integration test and runner #14
Conversation
Requires fix for #7 |
73f59d5
to
4ef0fa5
Compare
Codecov Report
@@ Coverage Diff @@
## main #14 +/- ##
==========================================
+ Coverage 73.37% 74.15% +0.77%
==========================================
Files 90 90
Lines 4819 4817 -2
==========================================
+ Hits 3536 3572 +36
+ Misses 1283 1245 -38
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
pareto_front_metrics = get_mean_metrics(pareto_front) | ||
|
||
quality_improved = pareto_front_metrics[0] < first_pop_metrics[0] | ||
complexity_improved = pareto_front_metrics[-1] < first_pop_metrics[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.
Здесь first_pop_metrics[1]
скорее всего имелось в виду first_pop_metrics[-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.
для двух значений они совпадают. но поправлю. а ты можешь воспроизвести? у меня не падает
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.
я пока залью, для отчетности. если смодешь воспроизвести -- поправлю
0d33d9d
to
995272a
Compare
No description provided.