VelocityCalculator - Code Metrics - Inspection of "[In progress] Feature/add grouping opt" - qbahn/grortir - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#43)

by Wojtek

created 2016-07-20 23:01 UTC

VelocityCalculator A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
wmc	3
c	1
b	0
f	1
dl	0
loc	27
rs	10

3 Methods

Rating	Name	Size	Complexity
A	calculate_initial_velocity()	4	1
A	__init__()	6	1
A	calculate()	8	1

import numpy as np
class SomeClass:
    def some_method(self):
        """Do x and return foo."""


class VelocityCalculator:
    """Calculate Velocity fo single stage.

    Attributes:
        current_velocity (list) : list of coordinates of speed for current stage
        """

    def __init__(self, stage):
        self.stage = stage
        self.C1 = 0.5

        self.C2 = 0.5

        self.w = 0.8

        self.current_velocity = None

    def calculate_initial_velocity(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        velocity = 0.02 * np.random.rand(len(self.stage.control_params)) - 0.01
        self.current_velocity = velocity
        return velocity

    def calculate(self,particle_best_position, leader_best_position):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        velocity = self.w * self.current_velocity
        + self.C1 * np.random.rand(len(self.stage.control_params)) * (

            leader_best_position - self.stage.control_params)
        + self.C2 * np.random.rand(len(self.stage.control_params)) * (

            particle_best_position - self.stage.control_params)
        self.current_velocity = velocity
        return velocity


Pull Request — master (#43)

VelocityCalculator A

Complexity

Size/Duplication

Importance

3 Methods

1. Missing Dependencies

2. Missing init.py files

1			import numpy as np
			0 ignored issues – show Coding Style introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report This module should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history... Configuration introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report The import `numpy` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
2
3
4			class VelocityCalculator:
5			"""Calculate Velocity fo single stage.
6
7			Attributes:
8			current_velocity (list) : list of coordinates of speed for current stage
9			"""
10
11			def __init__(self, stage):
12			self.stage = stage
13			self.C1 = 0.5
			0 ignored issues – show Coding Style Naming introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report The name `C1` does not conform to the attribute naming conventions (`[a-z_][a-z0-9_]{2,30}$`). 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.C2 = 0.5
			0 ignored issues – show Coding Style Naming introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report The name `C2` does not conform to the attribute naming conventions (`[a-z_][a-z0-9_]{2,30}$`). 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...
15			self.w = 0.8
			0 ignored issues – show Coding Style Naming introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report The name `w` does not conform to the attribute naming conventions (`[a-z_][a-z0-9_]{2,30}$`). 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.current_velocity = None
17
18			def calculate_initial_velocity(self):
			0 ignored issues – show Coding Style introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
19			velocity = 0.02 * np.random.rand(len(self.stage.control_params)) - 0.01
20			self.current_velocity = velocity
21			return velocity
22
23			def calculate(self,particle_best_position, leader_best_position):
			0 ignored issues – show Coding Style introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report Exactly one space required after comma def calculate(self,particle_best_position, leader_best_position): ^ Loading history... Coding Style introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
24			velocity = self.w * self.current_velocity
25			+ self.C1 * np.random.rand(len(self.stage.control_params)) * (
			0 ignored issues – show Unused Code Bug introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report The expression `((+self.C1) * (np.random....stage.control_params))` does not seem to have sideeffects and its result is not used. If a expression has no sideeffects (any lasting effect after it has been called) and its return value is not used, this usually means that this code can be removed or that an assignment is missing. Loading history...
26			leader_best_position - self.stage.control_params)
27			+ self.C2 * np.random.rand(len(self.stage.control_params)) * (
			0 ignored issues – show Unused Code Bug introduced 2016-07-20 23:10 UTC by Report Bug Copy Issue Report The expression `((+self.C2) * (np.random....stage.control_params))` does not seem to have sideeffects and its result is not used. If a expression has no sideeffects (any lasting effect after it has been called) and its return value is not used, this usually means that this code can be removed or that an assignment is missing. Loading history...
28			particle_best_position - self.stage.control_params)
29			self.current_velocity = velocity
30			return velocity
31

qbahn / grortir

Pull Request — master (#43)

VelocityCalculator A

Complexity

Size/Duplication

Importance

3 Methods

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files