Python.oshelper.Scheduler

Complexity

Total Complexity

1

Size/Duplication

Total Lines	51
Duplicated Lines	31.37 %

Importance

Changes

0

1 Function

Rating	Name	Duplication	Size	Complexity
A	DICOMMonitor()	0	11	1

import sys

This module should have a docstring.

import os

The import os seems to be unused.

import argparse
import getpass
import logging

logging.basicConfig(stream=sys.stdout, level=logging.INFO)



def DICOMMonitor():

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

    '''
    Monitor the DICOM folders of Orthanc output.
    :
    '''
    logger = logging.getLogger('DICOMMonitor')

The variable logger seems to be unused.

    default_locations = "example path"

The variable default_locations seems to be unused.

    default_monitoring_duration = 60 #seconds

The variable default_monitoring_duration seems to be unused.

    default_DCMTK = "DCMTK path"

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

The variable default_DCMTK seems to be unused.

    default_os = "ubuntu"

The variable default_os seems to be unused.

    #Check for all the new batch if they contain any of the known existing DICOM string.











if __name__ == '__main__':

This code seems to be duplicated in your project.

    parser = argparse.ArgumentParser(description=__doc__)

The name parser does not conform to the constant naming conventions ((([A-Z_][A-Z0-9_]*)|(__.*__))$).

    parser.add_argument('-u', '--user', dest='email', type=str, help='Username/Email used for login')

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

    parser.add_argument('-p', '--production', dest='production', action='store_true', help='Example of boolean arg')

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

    parser.add_argument('-o', '--option', dest='option', type=str, help='Example of str arg')

    parser.add_argument('file', metavar='file', type=str, help='Example of a positional argument')

    args = parser.parse_args()

The name args does not conform to the constant naming conventions ((([A-Z_][A-Z0-9_]*)|(__.*__))$).

    logger.info('--------------')

The variable logger does not seem to be defined.

Undefined variable 'logger'

    # Never ask for a password in command-line. Manually ask for it here
    password = getpass.getpass()

The name password does not conform to the constant naming conventions ((([A-Z_][A-Z0-9_]*)|(__.*__))$).

    logger.info('Hello World!')

Undefined variable 'logger'