Sign upLogin

KulikDM/pythresh

View on GitHub
Star
pythresh/thresholds/ocsvm.py

Summary

Maintainability
A
50 mins
Test Coverage

Function __init__ has 7 arguments (exceeds 4 allowed). Consider refactoring.
Open

    def __init__(self, model='sgd', degree='auto', gamma='auto',
Severity: Major
Found in pythresh/thresholds/ocsvm.py - About 50 mins to fix

    Method "__init__" has 8 parameters, which is greater than the 7 authorized.
    Open

        def __init__(self, model='sgd', degree='auto', gamma='auto',
                 criterion='bic', nu='auto', tol=1e-3, random_state=1234):
    Severity: Major
    Found in pythresh/thresholds/ocsvm.py by sonar-python

    A long parameter list can indicate that a new structure should be created to wrap the numerous parameters or that the function is doing too many things.

    Noncompliant Code Example

    With a maximum number of 4 parameters:

    def do_something(param1, param2, param3, param4, param5):
    ...

    Compliant Solution

    def do_something(param1, param2, param3, param4):
    ...

    There are no issues that match your filters.

    Category
    Status