hawc_hal.util

Complexity

Total Complexity

8

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes

0

3 Functions

Rating	Name	Duplication	Size	Complexity
A	cartesian_()	0	28	5
A	ra_to_longitude()	0	11	2
A	cartesian()	0	4	1

import numpy as np

This module should have a docstring.

Unable to import 'numpy'

def cartesian(arrays):

This function should have a docstring.

    return np.dstack(
        np.meshgrid(*arrays, indexing='ij')
        ).reshape(-1, len(arrays))


def cartesian_(arrays, out=None):
    """
    Generate a cartesian product of input arrays.
    """

    # A re-entrant function

    dtype = arrays[0].dtype

    n = np.prod([x.size for x in arrays])

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

    # This happen only the first iteration
    if out is None:

        out = np.zeros([n, len(arrays)], dtype=dtype)

    m = n / arrays[0].size

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

    out[:,0] = np.repeat(arrays[0], m)

Exactly one space required after comma

    if arrays[1:]:

        cartesian_(arrays[1:], out=out[0:m, 1:])

        for j in range(1, arrays[0].size):

            out[j*m:(j+1)*m, 1:] = out[0:m, 1:]

    return out


def ra_to_longitude(ra):

The name ra does not conform to the argument naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This function should have a docstring.

    if ra > 180.0:

        longitude = -180 + (ra - 180.0)

    else:

        longitude = ra

    return longitude