Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix plot_pareto #1023

Merged
merged 4 commits into from
Feb 10, 2023
Merged

Fix plot_pareto #1023

merged 4 commits into from
Feb 10, 2023

Conversation

YamLyubov
Copy link
Collaborator

@YamLyubov YamLyubov commented Jan 16, 2023

plot_pareto was using ApiParams.metric_to_compose attribute which was not set anywere. Now metric names are obtained from api_composer.

This solution might be refactored after closing #1038

Figure_1

@gkirgizov gkirgizov mentioned this pull request Jan 28, 2023
4 tasks
@nicl-nno nicl-nno requested a review from maypink February 9, 2023 10:24
@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.78%. Comparing base (df1e56c) to head (9797935).
Report is 139 commits behind head on master.

Files with missing lines Patch % Lines
fedot/api/api_utils/params.py 0.00% 1 Missing ⚠️
fedot/api/main.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1023      +/-   ##
==========================================
- Coverage   87.80%   87.78%   -0.02%     
==========================================
  Files         208      208              
  Lines       13968    13970       +2     
==========================================
  Hits        12264    12264              
- Misses       1704     1706       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@YamLyubov YamLyubov merged commit 8c4efb3 into master Feb 10, 2023
@YamLyubov YamLyubov deleted the fix-plot-pareto branch February 10, 2023 07:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants