Visualizer - Code Metrics - Inspection of "changed testing to py.test" - richlewis42/scikit-chem - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 97f98d...9ca1ad )

by Rich

created 2016-06-30 13:42 UTC

Visualizer A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	101
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
wmc	18
c	1
b	0
f	1
dl	0
loc	101
rs	10

12 Methods

Rating	Name	Size	Complexity
A	dpi()	6	1
A	update_smiles()	10	2
A	calculate()	5	1
A	typing()	2	1
A	update_dropdown()	5	1
A	current_smiles()	3	1
B	__init__()	29	2
A	current_bit()	3	1
A	initialize_ipython()	6	2
A	mol()	3	1
A	display()	2	1
A	plot()	5	1

#! /usr/bin/env python
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
#
# Copyright (C) 2016 Rich Lewis <[email protected]>
# License: 3-clause BSD

import matplotlib.pyplot as plt
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

from .. import descriptors
from .. import core
from .. import vis
from ipywidgets import Dropdown, Text, VBox, HBox, Valid, HTML
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from IPython import get_ipython
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from IPython.display import clear_output, display
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3


class Visualizer(object):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    def __init__(self, fper='morgan', smiles='c1ccccc1O', dpi=200):

        self.initialize_ipython()

        if isinstance(fper, str):
            self.fper = descriptors.get(fper)
        else:
            self.fper = fper

        self.smiles_input = Text(smiles, description='smiles')
        self.smiles_input.on_submit(self.update_smiles)
        self.smiles_input.observe(self.typing)

        self.valid = Valid(True)

        self.dropdown = Dropdown(options=[], description='bit')
        self.dropdown.observe(self.plot)

        self.dpi_input = Text(str(dpi), description='dpi')
        self.dpi_input.on_submit(self.plot)

        self.ui = VBox([

            HTML('<h2>Visualizer</h2>'),
            HBox([self.smiles_input, self.valid]),
            self.dropdown,
            self.dpi_input])

        self.update_smiles(None)
        self.display()

    def initialize_ipython(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        ipython = get_ipython()
        try:
            ipython.magic('matplotlib inline')
        except:

            pass

    def typing(self, _):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self.valid.visible = False

    @property
    def dpi(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            return int(self.dpi_input.value)
        except:

            return 50

    @dpi.setter
    def dpi(self, value):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self.dpi_input.value = str(value)

    def display(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        display(self.ui)

    def update_smiles(self, _):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            self._mol = core.Mol.from_smiles(self.smiles_input.value)

            self.valid.value = True
        except ValueError:
            self.valid.value = False
            return
        finally:
            self.valid.visible = True
        return self.calculate()

    def calculate(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        fp = self.fper.transform(self.mol)

        self.fp = fp[fp == 1].index
class Foo:
    def __init__(self, x=None):
        self.x = x
        self.fpg = self.fper.grad(self.mol).ix[self.fp]
class Foo:
    def __init__(self, x=None):
        self.x = x
        return self.update_dropdown()

    def update_dropdown(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self.dropdown.options.append(self.fp[0])
        self.dropdown.value = self.fp[0]
        self.dropdown.options = self.fp.tolist()
        return self.plot(self.dropdown.value)

    @property
    def mol(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self._mol

    @mol.setter
    def mol(self, mol):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self._mol = mol
class Foo:
    def __init__(self, x=None):
        self.x = x
        self.smiles_input.value = mol.to_smiles()
        self.calculate()

    @property
    def current_smiles(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self.smiles_input.value

    @property
    def current_bit(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return self.dropdown.value

    def plot(self, _):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        clear_output()
        plt.clf()
        plt.rcParams['savefig.dpi'] = self.dpi
        vis.plot_weights(self.mol, self.fpg.ix[self.current_bit], quality=4, ax=plt.gca())


1			#! /usr/bin/env python
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
2			#
3			# Copyright (C) 2016 Rich Lewis <[email protected]>
4			# License: 3-clause BSD
5
6			import matplotlib.pyplot as plt
			0 ignored issues – show Configuration introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The import `matplotlib.pyplot` 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...
7
8			from .. import descriptors
9			from .. import core
10			from .. import vis
11			from ipywidgets import Dropdown, Text, VBox, HBox, Valid, HTML
			0 ignored issues – show Configuration introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The import `ipywidgets` 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...
12			from IPython import get_ipython
			0 ignored issues – show Configuration introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The import `IPython` 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...
13			from IPython.display import clear_output, display
			0 ignored issues – show Configuration introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The import `IPython.display` 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...
14
15
16			class Visualizer(object):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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... best-practice introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report Too many instance attributes (9/7) Loading history...
17			def __init__(self, fper='morgan', smiles='c1ccccc1O', dpi=200):
18
19			self.initialize_ipython()
20
21			if isinstance(fper, str):
22			self.fper = descriptors.get(fper)
23			else:
24			self.fper = fper
25
26			self.smiles_input = Text(smiles, description='smiles')
27			self.smiles_input.on_submit(self.update_smiles)
28			self.smiles_input.observe(self.typing)
29
30			self.valid = Valid(True)
31
32			self.dropdown = Dropdown(options=[], description='bit')
33			self.dropdown.observe(self.plot)
34
35			self.dpi_input = Text(str(dpi), description='dpi')
36			self.dpi_input.on_submit(self.plot)
37
38			self.ui = VBox([
			0 ignored issues – show Coding Style Naming introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The name `ui` 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...
39			HTML('<h2>Visualizer</h2>'),
40			HBox([self.smiles_input, self.valid]),
41			self.dropdown,
42			self.dpi_input])
43
44			self.update_smiles(None)
45			self.display()
46
47			def initialize_ipython(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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... Coding Style introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report This method could be written as a function/class method. If a method does not access any attributes of the class, it could also be implemented as a function or static method. This can help improve readability. For example class Foo: def some_method(self, x, y): return x + y; could be written as class Foo: @classmethod def some_method(cls, x, y): return x + y; Loading history...
48			ipython = get_ipython()
49			try:
50			ipython.magic('matplotlib inline')
51			except:
			0 ignored issues – show Coding Style Best Practice introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report General except handlers without types should be used sparingly. Typically, you would use general except handlers when you intend to specifically handle all types of errors, f.e. when logging. Otherwise, such general error handlers can mask errors in your application that you want to know of. Loading history...
52			pass
53
54			def typing(self, _):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
55			self.valid.visible = False
56
57			@property
58			def dpi(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
59			try:
60			return int(self.dpi_input.value)
61			except:
			0 ignored issues – show Coding Style Best Practice introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report General except handlers without types should be used sparingly. Typically, you would use general except handlers when you intend to specifically handle all types of errors, f.e. when logging. Otherwise, such general error handlers can mask errors in your application that you want to know of. Loading history...
62			return 50
63
64			@dpi.setter
65			def dpi(self, value):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
66			self.dpi_input.value = str(value)
67
68			def display(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
69			display(self.ui)
70
71			def update_smiles(self, _):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
72			try:
73			self._mol = core.Mol.from_smiles(self.smiles_input.value)
			0 ignored issues – show Bug introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The Class `Mol` does not seem to have a member named `from_smiles`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
74			self.valid.value = True
75			except ValueError:
76			self.valid.value = False
77			return
78			finally:
79			self.valid.visible = True
80			return self.calculate()
81
82			def calculate(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
83			fp = self.fper.transform(self.mol)
			0 ignored issues – show Coding Style Naming introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The name `fp` does not conform to the variable 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... Bug introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The Instance of `str` does not seem to have a member named `transform`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
84			self.fp = fp[fp == 1].index
			0 ignored issues – show Coding Style Naming introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The name `fp` 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... Coding Style introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The attribute `fp` was defined outside `__init__`. It is generally a good practice to initialize all attributes to default values in the `__init__` method: class Foo: def __init__(self, x=None): self.x = x Loading history...
85			self.fpg = self.fper.grad(self.mol).ix[self.fp]
			0 ignored issues – show Bug introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The Instance of `str` does not seem to have a member named `grad`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history... Coding Style introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The attribute `fpg` was defined outside `__init__`. It is generally a good practice to initialize all attributes to default values in the `__init__` method: class Foo: def __init__(self, x=None): self.x = x Loading history...
86			return self.update_dropdown()
87
88			def update_dropdown(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
89			self.dropdown.options.append(self.fp[0])
90			self.dropdown.value = self.fp[0]
91			self.dropdown.options = self.fp.tolist()
92			return self.plot(self.dropdown.value)
93
94			@property
95			def mol(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
96			return self._mol
97
98			@mol.setter
99			def mol(self, mol):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
100			self._mol = mol
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 UTC by Report Bug Copy Issue Report The attribute `_mol` was defined outside `__init__`. It is generally a good practice to initialize all attributes to default values in the `__init__` method: class Foo: def __init__(self, x=None): self.x = x Loading history...
101			self.smiles_input.value = mol.to_smiles()
102			self.calculate()
103
104			@property
105			def current_smiles(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
106			return self.smiles_input.value
107
108			@property
109			def current_bit(self):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
110			return self.dropdown.value
111
112			def plot(self, _):
			0 ignored issues – show Coding Style introduced 2016-06-30 13:44 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...
113			clear_output()
114			plt.clf()
115			plt.rcParams['savefig.dpi'] = self.dpi
116			vis.plot_weights(self.mol, self.fpg.ix[self.current_bit], quality=4, ax=plt.gca())
117

Push — master ( 97f98d...9ca1ad )

Visualizer A

Complexity

Size/Duplication

Importance

12 Methods

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

richlewis42 / scikit-chem

Push — master ( 97f98d...9ca1ad )

Visualizer A

Complexity

Size/Duplication

Importance

12 Methods

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files

2. Missing init.py files

2. Missing init.py files

2. Missing init.py files