PositionUpdater - 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

PositionUpdater A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
wmc	7
c	2
b	0
f	1
dl	0
loc	22
rs	10

4 Methods

Rating	Name	Size	Complexity
A	__init__()	4	1
A	update_position()	3	1
A	set_initial_control_params()	5	1
A	_fix_coordinates()	6	4

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


class PositionUpdater:
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    def __init__(self, stage):
        self.stage = stage
        self.lower_bounds = np.asarray(self.stage.lower_bounds)
        self.upper_bounds = np.asarray(self.stage.upper_bounds)

    def set_initial_control_params(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        random = np.random.rand(len(self.stage.control_params))
        delta = self.upper_bounds - self.lower_bounds
        control_params = self.lower_bounds + random * delta
        self.stage.control_params = control_params.tolist()

    def update_position(self, velocity):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self.stage.control_params = self.stage.control_params + velocity
        self._fix_coordinates()

    def _fix_coordinates(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        for i in range(len(self.stage.control_params)):
            if self.stage.control_params[i] > self.stage.upper_bounds[i]:
                self.stage.control_params[i] = self.stage.upper_bounds[i]
            elif self.stage.control_params[i] < self.stage.lower_bounds[i]:
                self.stage.control_params[i] = self.stage.lower_bounds[i]


Pull Request — master (#43)

PositionUpdater A

Complexity

Size/Duplication

Importance

4 Methods

1. Missing Dependencies

2. Missing init.py files

1			import numpy as np
			0 ignored issues – show Coding Style introduced 2016-07-16 00:25 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-16 00:25 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 PositionUpdater:
			0 ignored issues – show Coding Style introduced 2016-07-16 00:25 UTC by Report Bug Copy Issue Report This class 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...
5			def __init__(self, stage):
6			self.stage = stage
7			self.lower_bounds = np.asarray(self.stage.lower_bounds)
8			self.upper_bounds = np.asarray(self.stage.upper_bounds)
9
10			def set_initial_control_params(self):
			0 ignored issues – show Coding Style introduced 2016-07-16 00:25 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...
11			random = np.random.rand(len(self.stage.control_params))
12			delta = self.upper_bounds - self.lower_bounds
13			control_params = self.lower_bounds + random * delta
14			self.stage.control_params = control_params.tolist()
15
16			def update_position(self, velocity):
			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...
17			self.stage.control_params = self.stage.control_params + velocity
18			self._fix_coordinates()
19
20			def _fix_coordinates(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...
21			for i in range(len(self.stage.control_params)):
22			if self.stage.control_params[i] > self.stage.upper_bounds[i]:
23			self.stage.control_params[i] = self.stage.upper_bounds[i]
24			elif self.stage.control_params[i] < self.stage.lower_bounds[i]:
25			self.stage.control_params[i] = self.stage.lower_bounds[i]
26

qbahn / grortir

Pull Request — master (#43)

PositionUpdater A

Complexity

Size/Duplication

Importance

4 Methods

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files