aimclub / FEDOT

Automated modeling and machine learning framework FEDOT
https://fedot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
619 stars 84 forks source link

Metrics fixes, tests, refactoring, typing improvement #1214

Closed MorrisNein closed 7 months ago

MorrisNein commented 7 months ago

Changes:

pep8speaks commented 7 months ago

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

Line 277:121: E501 line too long (123 > 120 characters)

Line 104:52: F811 redefinition of unused 'data_setup' from line 14

Comment last updated at 2023-11-29 18:17:04 UTC
codecov[bot] commented 7 months ago

Codecov Report

Attention: 30 lines in your changes are missing coverage. Please review.

Comparison is base (9d9469f) 79.27% compared to head (e1d6f84) 79.42%. Report is 1 commits behind head on master.

Files Patch % Lines
fedot/core/operations/atomized_model.py 0.00% 29 Missing :warning:
fedot/core/composer/metrics.py 98.11% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1214 +/- ## ========================================== + Coverage 79.27% 79.42% +0.14% ========================================== Files 145 145 Lines 10048 10045 -3 ========================================== + Hits 7966 7978 +12 + Misses 2082 2067 -15 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

MorrisNein commented 7 months ago

@kasyanovse, вынес проблему с бинарной классификацией в отдельную issue.

А пока что вынес этот случай в менее строгий тест.