tracking_policy_agendas.classifiers.meta_sklearn_clf.MetaSkLearnClf.save_model() - Code Metrics - Inspection of "Test and coverage" - MohammadForouhesh/tracking-policy-agendas - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Retried Inspection

Pull Request — main (#5)

by Mohammad

created 2022-03-13 12:21 UTC

MetaSkLearnClf.save_model() A

↳ Parent: tracking_policy_agendas.classifiers.meta_sklearn_clf

Complexity

Conditions

Size

Total Lines	8
Code Lines	8

Duplication

Lines	8
Ratio	100 %

Importance

Changes

Metric	Value
eloc	8
dl	8
loc	8
rs	10
c	0
b	0
f	0
cc	4
nop	2

import os

import pickle
from .meta_clf import MetaClf


class MetaSkLearnClf(MetaClf):

    def __init__(self, classifier_instance, **kwargs):
        super().__init__(classifier_instance=classifier_instance, **kwargs)

    def load_model(self, load_path: str):
        loading_prep = lambda string: f'model_dir/{load_path}/{string}'
        self.emb.load(loading_prep('emb.pkl'))
        with open(loading_prep('model.pkl'), 'rb') as f:

            self.scaler = pickle.load(f)
        with open(loading_prep('scaler.pkl'), 'rb') as f:

            self.scaler = pickle.load(f)

    def save_model(self, save_path: str):
        os.makedirs(f'model_dir/{save_path}', exist_ok=True)
        saving_prep = lambda string: f'model_dir/{save_path}/{string}'
        self.emb.save(saving_prep('emb.pkl'))
        with open(saving_prep('scaler.pkl'), 'wb') as f:

            pickle.dump(self.scaler, f, pickle.HIGHEST_PROTOCOL)
        with open(saving_prep('model.pkl'), 'wb') as f:

            pickle.dump(self.clf, f, pickle.HIGHEST_PROTOCOL)


1		import os
		0 ignored issues – show introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report Missing module docstring Loading history...
2		import pickle
3		from .meta_clf import MetaClf
4
5
6	View Code Duplication	class MetaSkLearnClf(MetaClf):
		0 ignored issues – show introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report Missing class docstring Loading history... Duplication introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
7		def __init__(self, classifier_instance, **kwargs):
8		super().__init__(classifier_instance=classifier_instance, **kwargs)
9
10		def load_model(self, load_path: str):
11		loading_prep = lambda string: f'model_dir/{load_path}/{string}'
12		self.emb.load(loading_prep('emb.pkl'))
13		with open(loading_prep('model.pkl'), 'rb') as f:
		0 ignored issues – show Coding Style Naming introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report Variable name "f" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
14		self.scaler = pickle.load(f)
15		with open(loading_prep('scaler.pkl'), 'rb') as f:
		0 ignored issues – show Coding Style Naming introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report Variable name "f" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
16		self.scaler = pickle.load(f)
17
18		def save_model(self, save_path: str):
19		os.makedirs(f'model_dir/{save_path}', exist_ok=True)
20		saving_prep = lambda string: f'model_dir/{save_path}/{string}'
21		self.emb.save(saving_prep('emb.pkl'))
22		with open(saving_prep('scaler.pkl'), 'wb') as f:
		0 ignored issues – show Coding Style Naming introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report Variable name "f" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
23		pickle.dump(self.scaler, f, pickle.HIGHEST_PROTOCOL)
24		with open(saving_prep('model.pkl'), 'wb') as f:
		0 ignored issues – show Coding Style Naming introduced 2022-03-13 12:10 UTC by Report Bug Copy Issue Report Variable name "f" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
25		pickle.dump(self.clf, f, pickle.HIGHEST_PROTOCOL)
26

MohammadForouhesh / tracking-policy-agendas

Pull Request — main (#5)

MetaSkLearnClf.save_model() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like