tensorflow/models

View on GitHub

Showing 11,634 of 11,634 total issues

Avoid deeply nested control flow statements.
Open

          if prediction[i] in retrieval_solution[key]:
            num_correct += 1
          already_predicted.add(prediction[i])
Severity: Major
Found in research/delf/delf/python/datasets/google_landmarks_dataset/metrics.py - About 45 mins to fix

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

    def test_retrieval(datasets, net, epoch, writer=None, model_directory=None,
                       precompute_whitening=None, data_root='data', multiscale=[1.],
                       test_image_size=1024):
      """Testing step.
    
    
    Severity: Minor
    Found in research/delf/delf/python/training/global_features/train_utils.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

    Avoid deeply nested control flow statements.
    Open

              for image_id in row[1].split(' '):
                ground_truth_ids.append(image_id)
            else:

      Avoid deeply nested control flow statements.
      Open

                if tf.summary.should_record_summaries().numpy():
                  last_summary_step_value = global_step_value
                  last_summary_time = current_time
      
                # Print to console if running locally.
      Severity: Major
      Found in research/delf/delf/python/training/train.py - About 45 mins to fix

        Avoid deeply nested control flow statements.
        Open

                  if (last_summary_step_value is not None and
                      last_summary_time is not None):
                    tf.summary.scalar(
                        'global_steps_per_sec',
                        (global_step_value - last_summary_step_value) /
        Severity: Major
        Found in research/delf/delf/python/training/train.py - About 45 mins to fix

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

          def main(argv):
            if len(argv) > 1:
              raise RuntimeError('Too many command-line arguments.')
          
            # Parse dataset to obtain query/index images, and ground-truth.
          Severity: Minor
          Found in research/delf/delf/python/delg/perform_retrieval.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 _build_train_tfrecord_dataset has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
          Open

          def _build_train_tfrecord_dataset(csv_path,
                                            clean_csv_path,
                                            image_dir,
                                            generate_train_validation_splits,
                                            validation_split_size,
          Severity: Minor
          Found in research/delf/delf/python/training/build_image_dataset.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 __init__ has 6 arguments (exceeds 4 allowed). Consider refactoring.
          Open

            def __init__(self,
          Severity: Minor
          Found in research/delf/delf/python/training/model/export_global_model.py - About 45 mins to fix

            Avoid deeply nested control flow statements.
            Open

                        if i < self.current_line:
                          continue
                        self.current_line = i
            Severity: Major
            Found in research/cvt_text/corpus_processing/unlabeled_data.py - About 45 mins to fix

              Avoid deeply nested control flow statements.
              Open

                          if not self.legacy_mode or i == self.middle_frame_index:
                            self.smooth_loss += 1.0 / (2**s) * self.depth_smoothness(
                                self.disp[i][s], self.images[s][:, :, :, 3 * i:3 * (i + 1)])
              
              
              Severity: Major
              Found in research/vid2depth/model.py - About 45 mins to fix

                Avoid deeply nested control flow statements.
                Open

                            if output_stride is None:
                              factor = 1
                            else:
                              factor = nominal_stride // output_stride
                            output = resnet_utils.subsample(output, factor)
                Severity: Major
                Found in research/deeplab/core/resnet_v1_beta_test.py - About 45 mins to fix

                  Avoid deeply nested control flow statements.
                  Open

                              if i > j:
                                # Need to inverse egomotion when going back in sequence.
                                egomotion_mult *= -1
                              # For compatiblity with SfMLearner, interpret all egomotion vectors
                              # as pointing toward the middle frame.  Note that unlike SfMLearner,
                  Severity: Major
                  Found in research/vid2depth/model.py - About 45 mins to fix

                    Avoid deeply nested control flow statements.
                    Open

                              if backbone[cell_num] == 3 and nas_remove_os32_stride:
                                stride = 1
                              filter_scaling *= hparams.filter_scaling_rate
                    Severity: Major
                    Found in research/deeplab/core/nas_network.py - About 45 mins to fix

                      Avoid deeply nested control flow statements.
                      Open

                                  if len(words) < self.config.max_sentence_length:
                                    yield words
                            self.current_file = 0
                      Severity: Major
                      Found in research/cvt_text/corpus_processing/unlabeled_data.py - About 45 mins to fix

                        Avoid deeply nested control flow statements.
                        Open

                                  if model_options.aspp_with_concat_projection:
                                    concat_logits = slim.conv2d(
                                        concat_logits, depth, 1, scope=CONCAT_PROJECTION_SCOPE)
                                    concat_logits = slim.dropout(
                                        concat_logits,
                        Severity: Major
                        Found in research/deeplab/model.py - About 45 mins to fix

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

                            def _parse_function(self, example_proto):
                              """Function to parse the example proto.
                          
                              Args:
                                example_proto: Proto in the format of tf.Example.
                          Severity: Minor
                          Found in research/deeplab/datasets/data_generator.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

                          Avoid deeply nested control flow statements.
                          Open

                                    if (model_options.add_image_level_feature and
                                        model_options.aspp_with_squeeze_and_excitation):
                                      concat_logits *= image_feature
                          
                          
                          Severity: Major
                          Found in research/deeplab/model.py - About 45 mins to fix

                            Avoid deeply nested control flow statements.
                            Open

                                        if self.ssim_weight > 0:
                                          self.ssim_error[s][key] = self.ssim(self.warped_image[s][key],
                                                                              target)
                                          # TODO(rezama): This should be min_pool2d().
                                          ssim_mask = slim.avg_pool2d(self.warp_mask[s][key], 3, 1, 'VALID')
                            Severity: Major
                            Found in research/vid2depth/model.py - About 45 mins to fix

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

                              def _decoder_with_sum_merge(decoder_features_list,
                              Severity: Minor
                              Found in research/deeplab/model.py - About 45 mins to fix

                                Avoid deeply nested control flow statements.
                                Open

                                          if example:
                                            s = example['folder_name']
                                            frame = example['file_name']
                                            if np.random.random() < 0.1:
                                              val_f.write('%s %s\n' % (s, frame))
                                Severity: Major
                                Found in research/vid2depth/dataset/gen_data.py - About 45 mins to fix
                                  Severity
                                  Category
                                  Status
                                  Source
                                  Language