BursiAmesConverter

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Test Coverage

Coverage

11.76%

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	__init__()	0	20	2

#! /usr/bin/env python

This module should have a docstring.

#
# Copyright (C) 2016 Rich Lewis <rl403@cam.ac.uk>
# License: 3-clause BSD

import os
import zipfile
import logging

LOGGER = logging.getLogger(__name__)

import numpy as np

The import numpy could not be resolved.

from ... import io

from .base import Converter, default_pipeline, contiguous_order
from ...cross_validation import SimThresholdSplit

class BursiAmesConverter(Converter):

This class should have a docstring.

    def __init__(self, directory, output_directory, output_filename='bursi_ames.h5'):

The __init__ method of the super-class Converter is not called.

This function exceeds the maximum number of variables (17/15).

        zip_path = os.path.join(directory, 'cas_4337.zip')
        output_path = os.path.join(output_directory, output_filename)

        with zipfile.ZipFile(zip_path) as f:

The name f does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

            sdf_path = f.extract('cas_4337.sdf')

        data = io.read_sdf(sdf_path)
        data.index.name = 'batch'
        data['is_mutagen'] = (data['Ames test categorisation'] == 'mutagen').astype(np.uint8)
        ms, y = data.structure, data.is_mutagen

The name ms does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

The name y does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

        pipeline = default_pipeline()
        ms, y = pipeline.transform_filter(ms, y)

The name ms does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

The name y does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

        cv = SimThresholdSplit(min_threshold=0.6, n_jobs=-1).fit(ms)

The name cv does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

        train, valid, test = cv.split((70, 15, 15))
        (ms, y, train, valid, test) = contiguous_order((ms, y, train, valid, test), (train, valid, test))

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

The name ms does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

The name y does not conform to the variable naming conventions ([a-z_][a-z0-9_]{2,30}$).

        splits = (('train', train), ('valid', valid), ('test', test))
        self.run(ms, y, output_path, splits=splits)

if __name__ == '__main__':
    logging.basicConfig(level=logging.INFO)
    LOGGER.info('Converting Bursi Ames Dataset...')
    BursiAmesConverter.convert()