MOEDescriptorCalculator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

16%

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	_transform_series()	0	17	3
A	__init__()	0	2	1
B	transform()	0	11	5

#! /usr/bin/env python
#
# Copyright (C) 2015-2016 Rich Lewis <rl403@cam.ac.uk>
# License: 3-clause BSD

"""
## skchem.descriptors.moe

Module specifying moe descriptors.
"""

class MOEDescriptorCalculator(object):

This class should have a docstring.

    def __init__(self):
        pass

    def transform(self, obj):

This method should have a docstring.

        if isinstance(obj, core.Mol):

Undefined variable 'core'

            return self._transform_series(pd.Series(obj)).iloc[0]

Undefined variable 'pd'

        elif isinstance(obj, pd.Series):

Undefined variable 'pd'

            return self._transform_series(obj)
        elif isinstance(obj, pd.DataFrame):

Undefined variable 'pd'

            return self._transform_series(obj.structure)
        elif isinstance(obj, (tuple, list)):
            return self._transform_series(obj)
        else:
            raise NotImplementedError

    def _transform_series(self, series):

        with tempfile.NamedTemporaryFile(suffix='.sdf') as in_file, tempfile.NamedTemporaryFile() as out_file:

This line is too long as per the coding-style (110/100).

Undefined variable 'tempfile'

            # write mols to file
            write_sdf(series, in_file.name)

Undefined variable 'write_sdf'

            args = ['mddesc', in_file.name, '-o', out_file.name] + self.index

The Instance of MOEDescriptorCalculator does not seem to have a member named index.

            LOGGER.info('Running: ' + ' '.join(args))

Undefined variable 'LOGGER'

            # call command line
            subprocess.call(args)

Undefined variable 'subprocess'

            try:
                finished = pd.read_table(out_file.name).set_index('id')

Undefined variable 'pd'

            except Exception:

Catching very general exceptions such as Exception is usually not recommended.

                finished = None
        finished.index = series.index
        return finished