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 multiobj tuning #1156

Merged
merged 4 commits into from
Aug 24, 2023
Merged

Fix multiobj tuning #1156

merged 4 commits into from
Aug 24, 2023

Conversation

YamLyubov
Copy link
Collaborator

  • TunerBuilder fixed to initialize multi objective tuner
  • Tests for multi objective tuning are added
  • Documentation for tuning is updated

@codecov
Copy link

codecov bot commented Aug 23, 2023

Codecov Report

Merging #1156 (15441e8) into master (97f31db) will decrease coverage by 0.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1156      +/-   ##
==========================================
- Coverage   78.31%   78.31%   -0.01%     
==========================================
  Files         130      130              
  Lines        9349     9355       +6     
==========================================
+ Hits         7322     7326       +4     
- Misses       2027     2029       +2     
Files Changed Coverage Δ
fedot/core/pipelines/tuning/tuner_builder.py 79.31% <66.66%> (-2.18%) ⬇️

... and 1 file with indirect coverage changes

@kasyanovse kasyanovse self-requested a review August 23, 2023 13:45
@aim-pep8-bot
Copy link

aim-pep8-bot commented Aug 24, 2023

Hello @YamLyubov! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-08-24 11:39:40 UTC

@YamLyubov YamLyubov merged commit cebd493 into master Aug 24, 2023
@YamLyubov YamLyubov deleted the fix-multiobj-tuning branch August 24, 2023 12:14
# 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.

4 participants