freqtrade/freqtrade

View on GitHub
freqtrade/optimize/hyperopt_tools.py

Summary

Maintainability
D
2 days
Test Coverage

File hyperopt_tools.py has 441 lines of code (exceeds 250 allowed). Consider refactoring.
Open

import logging
from copy import deepcopy
from datetime import datetime, timezone
from pathlib import Path
from typing import Any, Dict, Iterator, List, Optional, Tuple
Severity: Minor
Found in freqtrade/optimize/hyperopt_tools.py - About 6 hrs to fix

    Function get_result_table has a Cognitive Complexity of 39 (exceeds 5 allowed). Consider refactoring.
    Open

        def get_result_table(config: Config, results: list, total_epochs: int, highlight_best: bool,
                             print_colorized: bool, remove_header: int) -> str:
            """
            Log result table
            """
    Severity: Minor
    Found in freqtrade/optimize/hyperopt_tools.py - About 5 hrs to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Function _params_pretty_print has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.
    Open

        def _params_pretty_print(
                params, space: str, header: str, non_optimized: Optional[Dict] = None) -> None:
    
            if space in params or (non_optimized and space in non_optimized):
                space_params = HyperoptTools._space_params(params, space, 5)
    Severity: Minor
    Found in freqtrade/optimize/hyperopt_tools.py - About 2 hrs to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Function export_csv_file has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.
    Open

        def export_csv_file(config: Config, results: list, csv_file: str) -> None:
            """
            Log result to csv-file
            """
            if not results:
    Severity: Minor
    Found in freqtrade/optimize/hyperopt_tools.py - About 1 hr to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Function export_csv_file has 30 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        def export_csv_file(config: Config, results: list, csv_file: str) -> None:
            """
            Log result to csv-file
            """
            if not results:
    Severity: Minor
    Found in freqtrade/optimize/hyperopt_tools.py - About 1 hr to fix

      Function get_result_table has 27 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          def get_result_table(config: Config, results: list, total_epochs: int, highlight_best: bool,
                               print_colorized: bool, remove_header: int) -> str:
              """
              Log result table
              """
      Severity: Minor
      Found in freqtrade/optimize/hyperopt_tools.py - About 1 hr to fix

        Function get_result_table has 6 arguments (exceeds 4 allowed). Consider refactoring.
        Open

            def get_result_table(config: Config, results: list, total_epochs: int, highlight_best: bool,
        Severity: Minor
        Found in freqtrade/optimize/hyperopt_tools.py - About 45 mins to fix

          Function _params_update_for_json has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
          Open

              def _params_update_for_json(result_dict, params, non_optimized, space: str) -> None:
                  if (space in params) or (space in non_optimized):
                      space_params = HyperoptTools._space_params(params, space)
                      space_non_optimized = HyperoptTools._space_params(non_optimized, space)
                      all_space_params = space_params
          Severity: Minor
          Found in freqtrade/optimize/hyperopt_tools.py - About 45 mins to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function show_epoch_details has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
          Open

              def show_epoch_details(results, total_epochs: int, print_json: bool,
                                     no_header: bool = False, header_str: Optional[str] = None) -> None:
                  """
                  Display details of the hyperopt result
                  """
          Severity: Minor
          Found in freqtrade/optimize/hyperopt_tools.py - About 25 mins to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          There are no issues that match your filters.

          Category
          Status