aimclub / Fedot.Industrial

Python framework for automated time series classification, regression and forecasting
https://fedotindustrial.readthedocs.io
BSD 3-Clause "New" or "Revised" License
84 stars 7 forks source link

Improvement of Unit and Integration tests #116

Closed technocreep closed 4 months ago

technocreep commented 7 months ago

TO DO:

pep8speaks commented 7 months ago

Thanks for update, @technocreep!

Line 1:1: F401 'fedot as fedot_api' imported but unused Line 1:26: W292 no newline at end of file

Line 361:17: F541 f-string is missing placeholders

Line 38:19: W292 no newline at end of file

Line 92:13: E722 do not use bare 'except' Line 266:13: E722 do not use bare 'except' Line 404:121: E501 line too long (122 > 120 characters) Line 482:13: E722 do not use bare 'except'

Line 16:1: F403 'from fedot_ind.core.metrics.metrics_implementation import ' used; unable to detect undefined names Line 16:1: F401 'fedot_ind.core.metrics.metrics_implementation.' imported but unused

Line 74:47: F632 use ==/!= to compare constant literals (str, bytes, int, float, tuple)

Line 8:1: F403 'from fedot_ind.core.metrics.metrics_implementation import *' used; unable to detect undefined names Line 54:46: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation Line 69:24: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation Line 70:17: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation Line 76:33: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation Line 84:48: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation Line 94:50: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation Line 102:35: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation

Line 118:9: E704 multiple statements on one line (def) Line 121:9: E704 multiple statements on one line (def) Line 125:9: E704 multiple statements on one line (def) Line 130:9: E704 multiple statements on one line (def) Line 144:9: E704 multiple statements on one line (def) Line 165:9: E704 multiple statements on one line (def)

Line 14:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names Line 67:36: F405 'PCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 68:41: F405 'FastICAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 69:46: F405 'PolyFeaturesImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 125:20: F405 'ScalingImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 126:26: F405 'NormalizationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 128:30: F405 'ImputationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 130:23: F405 'KernelPCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations Line 239:32: E131 continuation line unaligned for hanging indent Line 254:62: W292 no newline at end of file

Line 60:111: E231 missing whitespace after ',' Line 821:36: W292 no newline at end of file

Line 68:30: F541 f-string is missing placeholders Line 94:26: F541 f-string is missing placeholders Line 220:25: F841 local variable 'problem_name' is assigned to but never used Line 260:29: F841 local variable 'univariate' is assigned to but never used Line 295:25: F841 local variable 'class_label_list' is assigned to but never used Line 339:59: E126 continuation line over-indented for hanging indent Line 468:53: W503 line break before binary operator Line 479:53: W503 line break before binary operator Line 518:53: W503 line break before binary operator Line 613:68: E126 continuation line over-indented for hanging indent Line 613:121: E501 line too long (143 > 120 characters) Line 641:60: E126 continuation line over-indented for hanging indent Line 687:33: F841 local variable 'ex' is assigned to but never used Line 694:33: F841 local variable 'ex' is assigned to but never used Line 703:45: E126 continuation line over-indented for hanging indent

Line 118:9: F841 local variable 'time_index' is assigned to but never used

Line 140:5: F841 local variable 'labels' is assigned to but never used Line 141:5: F841 local variable 'probs' is assigned to but never used Line 145:5: F841 local variable 'curr_backend' is assigned to but never used Line 151:29: W292 no newline at end of file Line 151:29: W292 no newline at end of file Line 151:29: W292 no newline at end of file

Line 1:1: F401 'fedot_ind.core.models.automl.fedot_implementation.FedotClassificationImplementation' imported but unused Line 1:1: F401 'fedot_ind.core.models.automl.fedot_implementation.FedotRegressionImplementation' imported but unused Line 1:121: E501 line too long (126 > 120 characters) Line 11:31: W292 no newline at end of file Line 11:31: W292 no newline at end of file Line 11:31: W292 no newline at end of file

Comment last updated at 2024-02-15 13:05:54 UTC
codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.21%. Comparing base (9548d29) to head (35dc2ba). Report is 32 commits behind head on main.

:exclamation: Current head 35dc2ba differs from pull request most recent head 36e8ea4

Please upload reports for the commit 36e8ea4 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #116 +/- ## ========================================== + Coverage 56.67% 57.21% +0.53% ========================================== Files 104 101 -3 Lines 8935 8855 -80 ========================================== + Hits 5064 5066 +2 + Misses 3871 3789 -82 ``` | [Flag](https://app.codecov.io/gh/aimclub/Fedot.Industrial/pull/116/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aimclub) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/aimclub/Fedot.Industrial/pull/116/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aimclub) | `57.21% <100.00%> (+0.53%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aimclub#carryforward-flags-in-the-pull-request-comment) to find out more.

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