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

[FI-60] Unit and Integration tests improvement #145

Closed technocreep closed 3 months ago

technocreep commented 4 months ago

TO DO:

pep8speaks commented 4 months ago

Thanks for update, @technocreep!

Line 199:121: E501 line too long (200 > 120 characters)

Line 489:17: F541 f-string is missing placeholders Line 491:17: F541 f-string is missing placeholders

Line 132:30: F541 f-string is missing placeholders

Line 4:1: F403 'from fastai.callback.hook import *' used; unable to detect undefined names Line 133:14: F405 'hook_outputs' may be undefined, or defined from star imports: fastai.callback.hook Line 134:18: F405 'hook_outputs' may be undefined, or defined from star imports: fastai.callback.hook

Line 4:1: F403 'from fastai.layers import *' used; unable to detect undefined names Line 103:21: F405 'AdaptiveConcatPool1d' may be undefined, or defined from star imports: fastai.layers Line 155:40: F405 'LinBnDrop' may be undefined, or defined from star imports: fastai.layers

Line 196:5: E704 multiple statements on one line (def) Line 318:9: F901 'raise NotImplemented' should be 'raise NotImplementedError'

Line 89:121: E501 line too long (128 > 120 characters)

Line 17:121: E501 line too long (121 > 120 characters) Line 253:121: E501 line too long (126 > 120 characters)

Line 1:121: E501 line too long (137 > 120 characters)

Line 3:121: E501 line too long (191 > 120 characters)

Line 48:1: F811 redefinition of unused 'test_load_univariate_data' from line 33

Comment last updated at 2024-06-20 12:54:52 UTC
codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 77.12766% with 43 lines in your changes missing coverage. Please review.

Project coverage is 64.55%. Comparing base (d5ff39d) to head (1953c15). Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
fedot_ind/core/models/nn/network_impl/tst.py 18.18% 9 Missing :warning:
fedot_ind/tools/synthetic/ts_generator.py 0.00% 8 Missing :warning:
fedot_ind/api/utils/industrial_strategy.py 86.20% 4 Missing :warning:
...dot_ind/core/models/automl/fedot_implementation.py 50.00% 3 Missing :warning:
fedot_ind/tools/example_utils.py 85.71% 3 Missing :warning:
...rchitecture/postprocessing/optimisation_history.py 0.00% 2 Missing :warning:
.../core/architecture/preprocessing/data_convertor.py 71.42% 2 Missing :warning:
...re/models/nn/network_modules/layers/conv_layers.py 50.00% 2 Missing :warning:
fedot_ind/core/models/nn/network_modules/other.py 50.00% 2 Missing :warning:
..._ind/core/operation/IndustrialCachableOperation.py 0.00% 2 Missing :warning:
... and 6 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #145 +/- ## =========================================== + Coverage 48.91% 64.55% +15.64% =========================================== Files 109 106 -3 Lines 11116 11026 -90 =========================================== + Hits 5437 7118 +1681 + Misses 5679 3908 -1771 ``` | [Flag](https://app.codecov.io/gh/aimclub/Fedot.Industrial/pull/145/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/145/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aimclub) | `64.55% <77.12%> (+15.64%)` | :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.

notion-workspace[bot] commented 4 months ago

🧪 unit-Тесты