json_dump() - Code Metrics - Inspection of "added parallelism" - richlewis42/scikit-chem - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9b3def...87dea9 )

by Rich

created 2016-08-18 19:53 UTC

json_dump() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	10
ccs	0
cts	0
cp	0
rs	9.2
cc	4
crap	20

#! /usr/bin/env python
#
# Copyright (C) 2016 Rich Lewis <[email protected]>
# License: 3-clause BSD

"""
# skchem.utils.io

IO helper functions for skchem.
"""

import yaml
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
import json


def line_count(filename):

    """ Quickly count the number of lines in a file.

    Adapted from http://stackoverflow.com/questions/845058/how-to-get-line-count-cheaply-in-python

    Args:
        filename (str):
            The name of the file to count for.

    """

    f = open(filename, 'rb')

    lines = 0
    buf_size = 1024 * 1024
    read_f = f.read
    buf = read_f(buf_size)
    while buf:
        lines += buf.count(b'\n')
        buf = read_f(buf_size)
    return lines


def sdf_count(filename):

    """ Efficiently count molecules in an sdf file.

    Specifically, the function counts the number of times '$$$$' occurs at the
    start of lines in the file.

    Args:
        filename (str): The filename of the sdf file.

    Returns:
        int: the number of molecules in the file.
    """

    with open(filename, 'rb') as f:

        return sum(1 for l in f if l[:4] == b'$$$$')


def json_dump(obj, target=None):
    """ Write object as json to file or stream, or return as string. """

    if target is None:
        return json.dumps(obj)
    elif isinstance(target, str):
        with open(target, 'w') as f:

            json.dump(obj, f)
    else:
        json.dump(obj, target)


def yaml_dump(obj, target=None):
    """ Write object as yaml to file or stream, or return as string. """

    if isinstance(target, str):
        with open(target, 'w') as f:

            yaml.dump(obj, f, default_flow_style=False)
    else:
        return yaml.dump(obj, target, default_flow_style=False)


Push — master ( 9b3def...87dea9 )

json_dump() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing init.py files

1		#! /usr/bin/env python
2		#
3		# Copyright (C) 2016 Rich Lewis <[email protected]>
4		# License: 3-clause BSD
5
6	1	"""
7		# skchem.utils.io
8
9		IO helper functions for skchem.
10		"""
11
12		import yaml
		0 ignored issues – show Configuration introduced 2016-08-18 20:05 UTC by Report Bug Copy Issue Report The import `yaml` 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	1	import json
14
15
16		def line_count(filename):
17
18		""" Quickly count the number of lines in a file.
19
20		Adapted from http://stackoverflow.com/questions/845058/how-to-get-line-count-cheaply-in-python
21
22		Args:
23		filename (str):
24		The name of the file to count for.
25
26		"""
27
28		f = open(filename, 'rb')
		0 ignored issues – show Coding Style Naming introduced 2016-07-18 17:48 UTC by Report Bug Copy Issue Report The name `f` 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...
29		lines = 0
30		buf_size = 1024 * 1024
31		read_f = f.read
32		buf = read_f(buf_size)
33		while buf:
34		lines += buf.count(b'\n')
35		buf = read_f(buf_size)
36	1	return lines
37
38
39		def sdf_count(filename):
40
41		""" Efficiently count molecules in an sdf file.
42
43		Specifically, the function counts the number of times '$$$$' occurs at the
44		start of lines in the file.
45
46		Args:
47		filename (str): The filename of the sdf file.
48
49		Returns:
50		int: the number of molecules in the file.
51		"""
52
53		with open(filename, 'rb') as f:
		0 ignored issues – show Coding Style Naming introduced 2016-07-18 17:48 UTC by Report Bug Copy Issue Report The name `f` 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...
54		return sum(1 for l in f if l[:4] == b'$$$$')
55
56
57		def json_dump(obj, target=None):
58		""" Write object as json to file or stream, or return as string. """
59
60		if target is None:
61		return json.dumps(obj)
62		elif isinstance(target, str):
63		with open(target, 'w') as f:
		0 ignored issues – show Coding Style Naming introduced 2016-08-18 20:05 UTC by Report Bug Copy Issue Report The name `f` 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...
64		json.dump(obj, f)
65		else:
66		json.dump(obj, target)
67
68
69		def yaml_dump(obj, target=None):
70		""" Write object as yaml to file or stream, or return as string. """
71
72		if isinstance(target, str):
73		with open(target, 'w') as f:
		0 ignored issues – show Coding Style Naming introduced 2016-08-18 20:05 UTC by Report Bug Copy Issue Report The name `f` 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...
74		yaml.dump(obj, f, default_flow_style=False)
75		else:
76		return yaml.dump(obj, target, default_flow_style=False)
77

richlewis42 / scikit-chem

Push — master ( 9b3def...87dea9 )

json_dump() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files