Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

minor upgrades in wavelets #166

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

minor upgrades in wavelets #166

wants to merge 19 commits into from

Conversation

v1docq
Copy link
Collaborator

@v1docq v1docq commented Oct 21, 2024

No description provided.

@pep8speaks
Copy link

pep8speaks commented Oct 21, 2024

Thanks for update, @v1docq!

Line 72:26: F541 f-string is missing placeholders
Line 79:26: F541 f-string is missing placeholders
Line 85:26: F541 f-string is missing placeholders
Line 437:9: E704 multiple statements on one line (def)

Line 13:1: F403 'from fedot_ind.core.metrics.metrics_implementation import *' used; unable to detect undefined names
Line 25:32: F405 'Optional' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 25:41: F405 'OperationParameters' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 46:41: F405 'pd' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 53:20: F405 'pd' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 55:20: F405 'pd' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 57:52: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 67:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 70:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 83:39: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 87:19: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 87:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 88:19: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 88:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 91:51: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 91:91: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 97:53: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 120:54: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 155:66: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 158:51: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 159:38: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 160:20: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation

Line 38:5: E704 multiple statements on one line (def)
Line 58:5: E704 multiple statements on one line (def)
Line 87:5: E704 multiple statements on one line (def)

Line 196:121: E501 line too long (125 > 120 characters)

Line 62:9: E704 multiple statements on one line (def)
Line 64:9: E704 multiple statements on one line (def)

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

Line 86:86: E231 missing whitespace after ','

Line 12:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names
Line 97:20: F405 'ScalingImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 98:26: F405 'NormalizationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 100:30: F405 'ImputationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 102:23: F405 'KernelPCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 238:19: W503 line break before binary operator

Comment last updated at 2024-10-24 14:07:27 UTC

Copy link

codecov bot commented Oct 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.98%. Comparing base (25f5497) to head (ec69b4d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #166   +/-   ##
=======================================
  Coverage   65.98%   65.98%           
=======================================
  Files         119      119           
  Lines       11977    11978    +1     
=======================================
+ Hits         7903     7904    +1     
  Misses       4074     4074           
Flag Coverage Δ
unittests 65.98% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants