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

Updates after Neiry colab #165

Merged
merged 17 commits into from
Oct 17, 2024
Merged

Updates after Neiry colab #165

merged 17 commits into from
Oct 17, 2024

Conversation

v1docq
Copy link
Collaborator

@v1docq v1docq commented Oct 14, 2024

No description provided.

@v1docq v1docq requested a review from technocreep October 14, 2024 09:55
@pep8speaks
Copy link

pep8speaks commented Oct 14, 2024

Thanks for update, @v1docq!

Line 24:18: F821 undefined name 'calculate_regression_metric'
Line 31:18: F821 undefined name 'init_input_data'
Line 64:18: F821 undefined name 'calculate_regression_metric'
Line 91:18: F821 undefined name 'calculate_regression_metric'

Line 71:26: F541 f-string is missing placeholders
Line 78:26: F541 f-string is missing placeholders
Line 83: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 24:32: F405 'Optional' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 24:41: F405 'OperationParameters' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 40:41: F405 'pd' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 47:20: F405 'pd' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 49:20: F405 'pd' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 51:52: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 61:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 64:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 71:39: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 75:19: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 75:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 76:19: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 76:28: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 79:51: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 79:91: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 85:53: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 108:54: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 143:66: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 146:51: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 147:38: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 148:20: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation

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

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

Line 7:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names
Line 105:36: F405 'PCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 106:41: F405 'FastICAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 107:46: F405 'PolyFeaturesImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations

Line 10:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names
Line 11:121: E501 line too long (127 > 120 characters)
Line 96:20: F405 'ScalingImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 97:26: F405 'NormalizationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 99:30: F405 'ImputationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 101:23: F405 'KernelPCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 231:19: W503 line break before binary operator

Comment last updated at 2024-10-17 11:52:00 UTC

Copy link

codecov bot commented Oct 16, 2024

Codecov Report

Attention: Patch coverage is 66.81614% with 74 lines in your changes missing coverage. Please review.

Project coverage is 66.00%. Comparing base (6440157) to head (16cd8a3).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
fedot_ind/core/models/tabular/tabular_extractor.py 0.00% 42 Missing ⚠️
fedot_ind/api/main.py 78.84% 11 Missing ⚠️
fedot_ind/api/utils/api_init.py 86.58% 11 Missing ⚠️
fedot_ind/core/models/base_extractor.py 64.28% 5 Missing ⚠️
fedot_ind/api/utils/data.py 86.66% 2 Missing ⚠️
fedot_ind/core/repository/IndustrialDispatcher.py 0.00% 2 Missing ⚠️
fedot_ind/core/optimizer/IndustrialEvoOptimizer.py 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #165      +/-   ##
==========================================
- Coverage   67.05%   66.00%   -1.05%     
==========================================
  Files         117      119       +2     
  Lines       11885    11977      +92     
==========================================
- Hits         7970     7906      -64     
- Misses       3915     4071     +156     
Flag Coverage Δ
unittests 66.00% <66.81%> (-1.05%) ⬇️

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.

Comment on lines +16 to +21

Attributes:
window_size (int): size of window
stride (int): stride for window
var_threshold (float): threshold for variance

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лучше тут написать актуальную задачу генератора

def generate_features_from_ts(self,
input_data: InputData,
window_length: int = None) -> InputData:
feature_domain_models = [model for model in KERNEL_BASELINE_FEATURE_GENERATORS]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут видимо KERNEL_BASELINE_FEATURE_GENERATORS.values или что-то такое

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.keys()


def generate_features_from_ts(self,
input_data: InputData,
window_length: int = None) -> InputData:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вероятно тут возвращаемый тип лист

Comment on lines +65 to +72
def _transform(self, input_data: InputData) -> np.array:
"""
Method for feature generation for all series
"""

feature_list = self.generate_features_from_ts(input_data)
self.predict = self.create_feature_matrix(feature_list)
return self.predict if not self.reduce_dimension else self._reduce_dim(self.predict, input_data.target)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут с тайпхинтами что-то перепуталось, потому что приходит не инпут дата, а лист

Comment on lines +30 to +33
try:
is_invalid = mutation.__name__.__contains__('resample')
except Exception:
is_invalid = mutation.name.__contains__('resample')
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

просто интересно, почему мы дропаем ресемпл?

@v1docq v1docq merged commit 25f5497 into main Oct 17, 2024
3 of 4 checks passed
@v1docq v1docq deleted the neiry_example branch October 17, 2024 13:51
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.

4 participants