Sign upLogin

tensorflow/models

View on GitHub
Star
official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.py

Summary

Maintainability
C
1 day
Test Coverage

File panoptic_deeplab_merge.py has 461 lines of code (exceeds 250 allowed). Consider refactoring.
Open

# Copyright 2024 The TensorFlow Authors. All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
Severity: Minor
Found in official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.py - About 7 hrs to fix

    Function _merge_semantic_and_instance_maps has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.
    Open

    def _merge_semantic_and_instance_maps(
        semantic_prediction: tf.Tensor,
        instance_maps: tf.Tensor,
        thing_class_ids: tf.Tensor,
        label_divisor: int,
    Severity: Minor
    Found in official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.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 __init__ has 10 arguments (exceeds 4 allowed). Consider refactoring.
    Open

      def __init__(
    Severity: Major
    Found in official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.py - About 1 hr to fix

      Function _get_panoptic_predictions has 10 arguments (exceeds 4 allowed). Consider refactoring.
      Open

      def _get_panoptic_predictions(
      Severity: Major
      Found in official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.py - About 1 hr to fix

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

        def _get_instances_from_heatmap_and_offset(
        Severity: Major
        Found in official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.py - About 50 mins to fix

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

          def _merge_semantic_and_instance_maps(
          Severity: Minor
          Found in official/projects/panoptic/modeling/layers/panoptic_deeplab_merge.py - About 45 mins to fix

            There are no issues that match your filters.

            Category
            Status