fixed test can build files. (#49)

modules/canbus/vehicle/lincoln/BUILD

@@ -89,4 +89,3 @@ cc_test(
 )
 
 cpplint()
-

modules/common/adapters/BUILD

@@ -46,8 +46,8 @@ cc_library(
         "//modules/common/time",
         "//modules/common/util",
         "@com_github_google_protobuf//:protobuf",
-        "@ros//:ros_common",
         "@glog//:glog",
+        "@ros//:ros_common",
     ],
 )
 
@@ -64,8 +64,8 @@ cc_test(
         ":adapter",
         ":adapter_gflags",
         "//modules/localization/proto:localization_proto",
-        "@ros//:ros_common",
         "@gtest//:main",
+        "@ros//:ros_common",
     ],
 )
 

modules/common/math/BUILD

@@ -279,7 +279,7 @@ cc_library(
     ],
     deps = [
         "//modules/common:log",
-        "//modules/common/math/qp_solver:qp_solver",
+        "//modules/common/math/qp_solver",
         "//modules/common/math/qp_solver:active_set_qp_solver",
         "@eigen//:eigen",
     ],
@@ -293,7 +293,7 @@ cc_test(
     ],
     deps = [
         ":mpc",
-        "//modules/common/math/qp_solver:qp_solver",
+        "//modules/common/math/qp_solver",
         "//modules/common/math/qp_solver:active_set_qp_solver",
         "@gtest//:main",
     ],

modules/common/math/qp_solver/BUILD

@@ -38,10 +38,10 @@ cc_library(
     ],
     deps = [
         ":qp_solver",
+        "//modules/common:log",
         "//modules/common/math/qp_solver:qp_solver_gflags",
         "@eigen//:eigen",
         "@qp_oases//:qp_oases",
-        "//modules/common:log",
     ],
 )
 

modules/common/status/status.h

@@ -84,7 +84,9 @@ class Status {
   /**
    * @brief defines the logic of testing if two Status are equal
    */
-  bool operator==(const Status &rh) const { return (this->code_ == rh.code_); }
+  bool operator==(const Status &rh) const {
+    return (this->code_ == rh.code_) && (this->msg_ == rh.msg_);
+  }
 
   /**
    * @brief defines the logic of testing if two Status are unequal

modules/decision/proto/BUILD

@@ -16,7 +16,7 @@ cc_proto_library(
 py_proto_compile(
     name = "decision_proto_pylib",
     protos = [
-        "decision.proto"
+        "decision.proto",
     ],
     deps = [
         "//modules/common/proto:common_proto_pylib",

modules/dreamview/backend/BUILD

@@ -4,16 +4,16 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "backend",
-    hdrs = [
-        "dreamview.h",
-    ],
     srcs = [
         "dreamview.cc",
         "main.cc",
     ],
+    hdrs = [
+        "dreamview.h",
+    ],
     deps = [
-        "//modules/common/adapters:adapter_manager",
         "//modules/common:apollo_app",
+        "//modules/common/adapters:adapter_manager",
         "//modules/dreamview/backend/simulation_world:simulation_world_updater",
         "//modules/dreamview/backend/websocket",
         "@civetweb//:civetweb++",

modules/dreamview/backend/map/BUILD

@@ -4,14 +4,14 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "map_service",
-    hdrs = [
-        "map_service.h"
-    ],
     srcs = [
-        "map_service.cc"
+        "map_service.cc",
+    ],
+    hdrs = [
+        "map_service.h",
     ],
     deps = [
-        "//modules/map/hdmap:hdmap",
+        "//modules/map/hdmap",
         "//third_party/json",
         "@glog//:glog",
     ],

modules/dreamview/backend/simulation_world/BUILD

@@ -73,12 +73,12 @@ cc_test(
 
 cc_library(
     name = "simulation_world_updater",
-    hdrs = [
-        "simulation_world_updater.h"
-    ],
     srcs = [
         "simulation_world_updater.cc",
     ],
+    hdrs = [
+        "simulation_world_updater.h",
+    ],
     deps = [
         ":simulation_world_service",
         "//modules/dreamview/backend/common:dreamview_gflags",

modules/map/hdmap/adapter/BUILD

@@ -5,36 +5,36 @@ package(default_visibility = ["//visibility:public"])
 cc_library(
     name = "opendrive_adapter",
     srcs = [
+        "coordinate_convert_tool.cc",
+        "proto_reorganization.cc",
         "xml_parser/header_xml_parser.cc",
-        "xml_parser/objects_xml_parser.cc",
-        "xml_parser/lanes_xml_parser.cc",
         "xml_parser/junctions_xml_parser.cc",
+        "xml_parser/lanes_xml_parser.cc",
+        "xml_parser/objects_xml_parser.cc",
         "xml_parser/roads_xml_parser.cc",
         "xml_parser/signals_xml_parser.cc",
         "xml_parser/util_xml_parser.cc",
-        "coordinate_convert_tool.cc",
-        "proto_reorganization.cc",
     ],
     hdrs = [
+        "coordinate_convert_tool.h",
+        "proto_reorganization.h",
         "xml_parser/common_define.h",
-        "xml_parser/status.h",
         "xml_parser/header_xml_parser.h",
-        "xml_parser/objects_xml_parser.h",
-        "xml_parser/lanes_xml_parser.h",
         "xml_parser/junctions_xml_parser.h",
+        "xml_parser/lanes_xml_parser.h",
+        "xml_parser/objects_xml_parser.h",
         "xml_parser/roads_xml_parser.h",
         "xml_parser/signals_xml_parser.h",
+        "xml_parser/status.h",
         "xml_parser/util_xml_parser.h",
-        "coordinate_convert_tool.h",
-        "proto_reorganization.h",
     ],
     deps = [
-        "@glog//:glog",
         "//modules/common/math",
+        "//modules/common/status",
+        "//modules/map/proto:map_proto",
+        "@glog//:glog",
         "@proj4//:proj4",
         "@tinyxml2//:tinyxml2",
-        "//modules/map/proto:map_proto",
-        "//modules/common/status:status",
     ],
 )
 

modules/map/proto/BUILD

@@ -17,11 +17,11 @@ cc_proto_library(
         "map_lane.proto",
         "map_overlap.proto",
         "map_parking_space.proto",
+        "map_road.proto",
         "map_signal.proto",
         "map_speed_bump.proto",
         "map_stop_sign.proto",
         "map_yield_sign.proto",
-        "map_road.proto",
         "routing.proto",
     ],
     deps = [

modules/perception/BUILD

@@ -7,8 +7,8 @@ cc_library(
     srcs = ["perception.cc"],
     hdrs = ["perception.h"],
     deps = [
-        "//modules/common:log",
         "//modules/common:apollo_app",
+        "//modules/common:log",
         "//modules/common/adapters:adapter_manager",
         "//modules/perception/common:perception_common",
         "//modules/perception/obstacle/onboard:perception_obstacle_lidar_process",

modules/perception/common/BUILD

@@ -11,7 +11,7 @@ cc_library(
         "*.h",
     ]),
     deps = [
-        "//modules/common:common"
+        "//modules/common",
     ],
 )
 

modules/perception/lib/base/BUILD

@@ -18,7 +18,7 @@ cc_library(
         "-lboost_system",
     ],
     deps = [
-        "//modules/common:common",
+        "//modules/common",
         "//modules/common:log",
     ],
 )
@@ -36,5 +36,5 @@ cc_test(
         "@gtest//:main",
     ],
 )
-  
+
 cpplint()

modules/perception/lib/config_manager/BUILD

@@ -4,20 +4,20 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "config_manager",
-    srcs = ["config_manager.cc",],
-    hdrs = ["config_manager.h",],
+    srcs = ["config_manager.cc"],
+    hdrs = ["config_manager.h"],
     deps = [
         "//modules/common:log",
-        "//modules/perception/lib/base:base",
-        "//modules/perception/lib/config_manager/proto:config_proto",
         "//modules/perception/common:perception_common",
+        "//modules/perception/lib/base",
+        "//modules/perception/lib/config_manager/proto:config_proto",
     ],
 )
 
 cc_test(
     name = "config_manager_test",
-    srcs = ["config_manager_test.cc",],
     size = "small",
+    srcs = ["config_manager_test.cc"],
     data = ["//modules/perception:perception_testdata"],
     deps = [
         ":config_manager",

modules/perception/lib/config_manager/proto/BUILD

@@ -5,7 +5,6 @@ load("@org_pubref_rules_protobuf//cpp:rules.bzl", "cc_proto_library")
 cc_proto_library(
     name = "config_proto",
     protos = [
-        "config_schema.proto"
-    ]
+        "config_schema.proto",
+    ],
 )
-

modules/perception/lib/pcl_util/BUILD

@@ -7,7 +7,7 @@ cc_library(
     srcs = [],
     hdrs = glob(["*.h"]),
     deps = [
-        "@pcl//:pcl"
+        "@pcl//:pcl",
     ],
 )
 

modules/perception/obstacle/base/BUILD

@@ -7,18 +7,18 @@ cc_library(
     srcs = [
         "object.cc",
         "types.cc",
-        ],
+    ],
     hdrs = [
+        "hdmap_struct.h",
         "object.h",
         "types.h",
-        "hdmap_struct.h",
-        ],
+    ],
     deps = [
-        "//modules/perception/lib/pcl_util:pcl_util",
+        "//modules/common",
+        "//modules/common/util:string_util",
         "//modules/perception/common:perception_common",
+        "//modules/perception/lib/pcl_util",
         "//modules/perception/proto:perception_proto",
-        "//modules/common:common",
-        "//modules/common/util:string_util",
         "@eigen//:eigen",
     ],
 )

modules/perception/obstacle/common/BUILD

@@ -6,15 +6,15 @@ cc_library(
     name = "perception_obstacle_common",
     srcs = [
         "geometry_util.cc",
-        ],
+    ],
     hdrs = [
-        "geometry_util.h",
         "convex_hullxy.h",
-        ],
+        "geometry_util.h",
+    ],
     deps = [
         "//modules/common:log",
-        "//modules/perception/lib/pcl_util:pcl_util",
-        "@eigen//:eigen"
+        "//modules/perception/lib/pcl_util",
+        "@eigen//:eigen",
     ],
 )
 
@@ -23,7 +23,7 @@ cc_test(
     size = "small",
     srcs = [
         "geometry_util_test.cc",
-        ],
+    ],
     deps = [
         ":perception_obstacle_common",
         "@gtest//:main",

modules/perception/obstacle/lidar/dummy/BUILD

@@ -5,17 +5,17 @@ package(default_visibility = ["//visibility:public"])
 cc_library(
     name = "perception_obstacle_lidar_dummy",
     srcs = [
-        "dummy_algorithms.cc"
+        "dummy_algorithms.cc",
     ],
     hdrs = [
-        "dummy_algorithms.h"
+        "dummy_algorithms.h",
     ],
     deps = [
         "//modules/common:log",
-        "//modules/perception/obstacle/lidar/interface:perception_obstacle_lidar_interface",
         "//modules/perception/obstacle/common:perception_obstacle_common",
-        "@pcl//:pcl",
+        "//modules/perception/obstacle/lidar/interface:perception_obstacle_lidar_interface",
         "@eigen//:eigen",
+        "@pcl//:pcl",
     ],
 )
 

modules/perception/obstacle/lidar/interface/BUILD

@@ -7,11 +7,11 @@ cc_library(
     srcs = [],
     hdrs = glob(["*.h"]),
     deps = [
-        "//modules/perception/lib/pcl_util:pcl_util",
-        "//modules/perception/lib/base:base",
+        "//modules/perception/lib/base",
+        "//modules/perception/lib/pcl_util",
         "//modules/perception/obstacle/base:perception_obstacle_base",
         "@eigen//:eigen",
-    ]
+    ],
 )
 
 cpplint()

modules/perception/obstacle/lidar/object_builder/min_box/BUILD

@@ -7,13 +7,13 @@ cc_library(
     srcs = ["min_box.cc"],
     hdrs = ["min_box.h"],
     deps = [
-        "//modules/perception/lib/base:base",
-        "//modules/perception/lib/pcl_util:pcl_util",
+        "//modules/common",
+        "//modules/common:log",
         "//modules/perception/common:perception_common",
+        "//modules/perception/lib/base",
+        "//modules/perception/lib/pcl_util",
         "//modules/perception/obstacle/common:perception_obstacle_common",
         "//modules/perception/obstacle/lidar/interface:perception_obstacle_lidar_interface",
-        "//modules/common:log",
-        "//modules/common:common",
         "@eigen//:eigen",
     ],
 )
@@ -30,12 +30,11 @@ cc_test(
     ],
     deps = [
         ":perception_obstacle_lidar_object_builder_min_box",
-        "//modules/perception/lib/pcl_util:pcl_util",
         "//modules/perception/common:perception_common",
+        "//modules/perception/lib/pcl_util",
         "//modules/perception/obstacle/common:perception_obstacle_common",
         "@gtest//:main",
     ],
 )
 
 cpplint()
-

modules/perception/obstacle/onboard/BUILD

@@ -7,13 +7,13 @@ cc_library(
     srcs = ["hdmap_input.cc"],
     hdrs = ["hdmap_input.h"],
     deps = [
-        "//modules/perception/lib/base:base",
-        "//modules/perception/lib/pcl_util:pcl_util",
+        "//modules/common",
+        "//modules/common:log",
+        "//modules/map/hdmap",
         "//modules/perception/common:perception_common",
+        "//modules/perception/lib/base",
+        "//modules/perception/lib/pcl_util",
         "//modules/perception/obstacle/base:perception_obstacle_base",
-        "//modules/common:log",
-        "//modules/common:common",
-        "//modules/map/hdmap:hdmap",
         "@eigen//:eigen",
     ],
 )
@@ -24,15 +24,15 @@ cc_library(
     hdrs = ["lidar_process.h"],
     deps = [
         ":perception_obstacle_hdmapinput",
-        "//modules/perception/lib/base:base",
-        "//modules/perception/lib/pcl_util:pcl_util",
-        "//modules/perception/lib/config_manager:config_manager",
+        "//modules/common",
+        "//modules/common:log",
         "//modules/perception/common:perception_common",
+        "//modules/perception/lib/base",
+        "//modules/perception/lib/config_manager",
+        "//modules/perception/lib/pcl_util",
         "//modules/perception/obstacle/base:perception_obstacle_base",
         "//modules/perception/obstacle/lidar/dummy:perception_obstacle_lidar_dummy",
         "//modules/perception/obstacle/lidar/interface:perception_obstacle_lidar_interface",
-        "//modules/common:log",
-        "//modules/common:common",
         "@eigen//:eigen",
         "@ros//:ros_common",
     ],

modules/planning/math/frame_conversion/BUILD

@@ -12,8 +12,8 @@ cc_library(
     ],
     deps = [
         "//modules/common:log",
-        "//modules/common/math:vec2d",
         "//modules/common/math:math_utils",
+        "//modules/common/math:vec2d",
     ],
 )
 

modules/planning/math/spiral_curve/BUILD

@@ -41,9 +41,9 @@ cc_library(
         ":spiral_curve",
         ":spiral_formula",
         "//modules/common:log",
+        "//modules/common/math",
         "//modules/common/proto:error_code_proto",
         "//modules/common/proto:path_point_proto",
-        "//modules/common/math:math",
         "@eigen//:eigen",
     ],
 )
@@ -60,8 +60,8 @@ cc_library(
         ":spiral_curve",
         ":spiral_formula",
         "//modules/common:log",
+        "//modules/common/math",
         "//modules/common/proto:path_point_proto",
-        "//modules/common/math:math",
         "@eigen//:eigen",
     ],
 )

modules/planning/optimizer/dp_poly_path/BUILD

@@ -19,7 +19,6 @@ cc_library(
         "trajectory_cost.h",
     ],
     deps = [
-        "@eigen//:eigen",
         "//modules/common/configs:vehicle_config_helper",
         "//modules/common/math",
         "//modules/common/status",
@@ -37,6 +36,7 @@ cc_library(
         "//modules/planning/math/curve1d:quintic_polynomial_curve1d",
         "//modules/planning/proto:dp_poly_path_config_proto",
         "//modules/planning/reference_line",
+        "@eigen//:eigen",
     ],
 )
 

modules/planning/optimizer/dp_st_speed/dp_st_speed_optimizer.cc

@@ -63,6 +63,7 @@ Status DpStSpeedOptimizer::Process(const PathData& path_data,
     AERROR << "Please call Init() before process DpStSpeedOptimizer.";
     return Status(ErrorCode::PLANNING_ERROR, "Not inited.");
   }
+
   // step 1 get boundaries
   double planning_distance = std::min(dp_st_speed_config_.total_path_length(),
                                       path_data.path().param_length());
@@ -85,13 +86,15 @@ Status DpStSpeedOptimizer::Process(const PathData& path_data,
 
   // step 2 perform graph search
   SpeedLimit speed_limit;
-  if (boundary_mapper_.get_speed_limits(
-          common::VehicleState::instance()->pose(),
-          DataCenter::instance()->map(), path_data, planning_distance,
-          dp_st_speed_config_.matrix_dimension_s(),
-          dp_st_speed_config_.max_speed(), &speed_limit) != Status::OK()) {
-    const std::string msg
-        = "Getting speed limits for dp st speed optimizer failed!";
+  if (!boundary_mapper_
+           .get_speed_limits(common::VehicleState::instance()->pose(),
+                             DataCenter::instance()->map(), path_data,
+                             planning_distance,
+                             dp_st_speed_config_.matrix_dimension_s(),
+                             dp_st_speed_config_.max_speed(), &speed_limit)
+           .ok()) {
+    const std::string msg =
+        "Getting speed limits for dp st speed optimizer failed!";
     AERROR << msg;
     return Status(ErrorCode::PLANNING_ERROR, msg);
   }

modules/planning/optimizer/qp_spline_path/BUILD

@@ -25,8 +25,8 @@ cc_library(
         "//modules/map/proto:map_proto",
         "//modules/planning/common:data_center",
         "//modules/planning/common:decision_data",
-        "//modules/planning/common:planning_data",
         "//modules/planning/common:obstacle",
+        "//modules/planning/common:planning_data",
         "//modules/planning/common/path:discretized_path",
         "//modules/planning/common/path:frenet_frame_path",
         "//modules/planning/common/path:path_data",

modules/planning/optimizer/qp_spline_st_speed/qp_spline_st_boundary_mapper.cc

@@ -82,13 +82,13 @@ Status QpSplineStBoundaryMapper::get_graph_boundary(
     ret =
         map_main_decision_stop(main_decision.stop(), reference_line,
                                planning_distance, planning_time, obs_boundary);
-    if (ret != Status::OK() && ret != Status(ErrorCode::PLANNING_SKIP)) {
+    if (!ret.ok() && ret.code() != ErrorCode::PLANNING_SKIP) {
       return Status(ErrorCode::PLANNING_ERROR);
     }
   } else if (main_decision.has_mission_complete()) {
     ret = map_mission_complete(reference_line, planning_distance, planning_time,
                                obs_boundary);
-    if (ret != Status::OK() && ret != Status(ErrorCode::PLANNING_SKIP)) {
+    if (!ret.ok() && ret.code() != ErrorCode::PLANNING_SKIP) {
       return Status(ErrorCode::PLANNING_ERROR);
     }
   }

modules/planning/optimizer/qp_spline_st_speed/qp_spline_st_speed_optimizer.cc

@@ -72,11 +72,12 @@ Status QpSplineStSpeedOptimizer::Process(const PathData& path_data,
 
   // step 1 get boundaries
   std::vector<StGraphBoundary> boundaries;
-  if (boundary_mapper_.get_graph_boundary(
-          init_point, *decision_data, path_data, reference_line,
-          qp_spline_st_speed_config_.total_path_length(),
-          qp_spline_st_speed_config_.total_time(),
-          &boundaries) != Status::OK()) {
+  if (!boundary_mapper_
+           .get_graph_boundary(
+               init_point, *decision_data, path_data, reference_line,
+               qp_spline_st_speed_config_.total_path_length(),
+               qp_spline_st_speed_config_.total_time(), &boundaries)
+           .ok()) {
     return Status(ErrorCode::PLANNING_ERROR,
                   "Mapping obstacle for dp st speed optimizer failed!");
   }

modules/prediction/BUILD

@@ -10,15 +10,15 @@ cc_library(
         "//modules/common",
         "//modules/common:apollo_app",
         "//modules/common:log",
-        "//modules/common/util:util",
         "//modules/common/adapters:adapter_manager",
+        "//modules/common/util",
+        "//modules/perception/proto:perception_proto",
+        "//modules/prediction/common:prediction_gflags",
         "//modules/prediction/container:container_manager",
         "//modules/prediction/evaluator:evaluator_manager",
         "//modules/prediction/predictor:predictor_manager",
-        "//modules/perception/proto:perception_proto",
-        "//modules/prediction/common:prediction_gflags",
-        "//modules/prediction/proto:prediction_proto",
         "//modules/prediction/proto:prediction_conf_proto",
+        "//modules/prediction/proto:prediction_proto",
     ],
 )
 
@@ -36,8 +36,8 @@ cc_binary(
     name = "prediction",
     srcs = ["main.cc"],
     deps = [
-        "//modules/prediction:prediction_lib",
         "//external:gflags",
+        "//modules/prediction:prediction_lib",
     ],
 )
 

modules/prediction/common/BUILD

@@ -21,18 +21,17 @@ cc_library(
 
 cc_library(
     name = "prediction_map",
-    hdrs = ["prediction_map.h"],
     srcs = ["prediction_map.cc"],
+    hdrs = ["prediction_map.h"],
     deps = [
-        "@eigen//:eigen",
-        "//modules/map/hdmap:hdmap",
         "//modules/common:macro",
-        "//modules/prediction/common:prediction_gflags",
-        "//modules/map/pnc_map:pnc_path",
         "//modules/common/math:linear_interpolation",
         "//modules/common/math:vec2d",
+        "//modules/map/hdmap",
+        "//modules/map/pnc_map:pnc_path",
+        "//modules/prediction/common:prediction_gflags",
+        "@eigen//:eigen",
     ],
-
 )
 
 cc_library(
@@ -40,10 +39,10 @@ cc_library(
     srcs = ["road_graph.cc"],
     hdrs = ["road_graph.h"],
     deps = [
-        "//modules/prediction/proto:lane_graph_proto",
-        "//modules/map/hdmap:hdmap",
         "//modules/common/proto:error_code_proto",
+        "//modules/map/hdmap",
         "//modules/prediction/common:prediction_map",
+        "//modules/prediction/proto:lane_graph_proto",
     ],
 )
 

modules/prediction/container/BUILD

@@ -7,19 +7,19 @@ cc_library(
     srcs = ["container_manager.cc"],
     hdrs = ["container_manager.h"],
     deps = [
+        "//modules/common:macro",
         "//modules/prediction/container/obstacles:obstacles_container",
         "//modules/prediction/container/pose:pose_container",
-        "//modules/common:macro",
-        "@glog//:glog"
-    ]
+        "@glog//:glog",
+    ],
 )
 
 cc_library(
     name = "container",
     hdrs = ["container.h"],
     deps = [
-        "@com_github_google_protobuf//:protobuf"
-    ]
+        "@com_github_google_protobuf//:protobuf",
+    ],
 )
 
-cpplint()
\ No newline at end of file
+cpplint()

modules/prediction/container/obstacles/BUILD

@@ -7,10 +7,10 @@ cc_library(
     srcs = ["obstacles_container.cc"],
     hdrs = ["obstacles_container.h"],
     deps = [
-        "//modules/prediction/container:container",
-        "//modules/prediction/container/obstacles:obstacle",
-        "//modules/common/util:lru_cache",
         "//modules/common/math:math_utils",
+        "//modules/common/util:lru_cache",
+        "//modules/prediction/container",
+        "//modules/prediction/container/obstacles:obstacle",
         "//modules/prediction/container/pose:pose_container",
     ],
 )
@@ -20,17 +20,17 @@ cc_library(
     srcs = ["obstacle.cc"],
     hdrs = ["obstacle.h"],
     deps = [
-        "@eigen//:eigen",
         "//modules/common:log",
         "//modules/common/math:kalman_filter",
-        "//modules/common/proto:error_code_proto",
         "//modules/common/math:math_utils",
+        "//modules/common/proto:error_code_proto",
+        "//modules/map/hdmap",
         "//modules/perception/proto:perception_proto",
-        "//modules/prediction/proto:feature_proto",
         "//modules/prediction/common:prediction_gflags",
         "//modules/prediction/common:prediction_map",
         "//modules/prediction/common:road_graph",
-        "//modules/map/hdmap:hdmap",
+        "//modules/prediction/proto:feature_proto",
+        "@eigen//:eigen",
     ],
 )
 

modules/prediction/container/pose/BUILD

@@ -7,10 +7,10 @@ cc_library(
     srcs = ["pose_container.cc"],
     hdrs = ["pose_container.h"],
     deps = [
-        "//modules/prediction/container:container",
-        "//modules/perception/proto:perception_proto",
         "//modules/localization/proto:localization_proto",
-    ]
+        "//modules/perception/proto:perception_proto",
+        "//modules/prediction/container",
+    ],
 )
 
-cpplint()
\ No newline at end of file
+cpplint()

modules/prediction/evaluator/BUILD

@@ -4,16 +4,16 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "evaluator_factory",
-    hdrs = ["evaluator_factory.h"],
     srcs = ["evaluator_factory.cc"],
+    hdrs = ["evaluator_factory.h"],
     deps = [
-        "//modules/prediction/evaluator:evaluator",
+        "//modules/common:macro",
+        "//modules/common/util:factory",
+        "//modules/prediction/evaluator",
         "//modules/prediction/evaluator/vehicle:mlp_evaluator",
         "//modules/prediction/proto:prediction_conf_proto",
-        "//modules/common/util:factory",
-        "//modules/common:macro",
         "@glog//:glog",
-    ]
+    ],
 )
 
 cc_library(
@@ -21,24 +21,24 @@ cc_library(
     srcs = ["evaluator_manager.cc"],
     hdrs = ["evaluator_manager.h"],
     deps = [
-        "//modules/prediction/evaluator:evaluator",
-        "//modules/prediction/evaluator:evaluator_factory",
+        "//modules/common:log",
+        "//modules/common:macro",
+        "//modules/perception/proto:perception_proto",
         "//modules/prediction/container:container_manager",
         "//modules/prediction/container/obstacles:obstacles_container",
+        "//modules/prediction/evaluator",
+        "//modules/prediction/evaluator:evaluator_factory",
         "//modules/prediction/proto:prediction_conf_proto",
-        "//modules/perception/proto:perception_proto",
-        "//modules/common:macro",
-        "//modules/common:log",
         "@glog//:glog",
-    ]
+    ],
 )
 
 cc_library(
     name = "evaluator",
     hdrs = ["evaluator.h"],
     deps = [
-        "//modules/prediction/container/obstacles:obstacle"
-    ]
+        "//modules/prediction/container/obstacles:obstacle",
+    ],
 )
 
 cpplint()

modules/prediction/evaluator/vehicle/BUILD

@@ -4,20 +4,20 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "mlp_evaluator",
-    hdrs = [
-        "mlp_evaluator.h",
-    ],
     srcs = [
         "mlp_evaluator.cc",
     ],
+    hdrs = [
+        "mlp_evaluator.h",
+    ],
     deps = [
-        "//modules/prediction/evaluator:evaluator",
-        "//modules/prediction/container/obstacles:obstacle",
-        "//modules/prediction/proto:lane_graph_proto",
-        "//modules/prediction/common:prediction_gflags",
         "//modules/common/math:math_utils",
-        "//modules/prediction/proto:fnn_vehicle_model_proto",
+        "//modules/prediction/common:prediction_gflags",
         "//modules/prediction/common:prediction_util",
+        "//modules/prediction/container/obstacles:obstacle",
+        "//modules/prediction/evaluator",
+        "//modules/prediction/proto:fnn_vehicle_model_proto",
+        "//modules/prediction/proto:lane_graph_proto",
     ],
 )
 

modules/prediction/predictor/BUILD

@@ -4,17 +4,17 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "predictor_factory",
-    hdrs = ["predictor_factory.h"],
     srcs = ["predictor_factory.cc"],
+    hdrs = ["predictor_factory.h"],
     deps = [
-        "//modules/prediction/predictor:predictor",
-        "//modules/prediction/predictor/vehicle:lane_sequence_predictor",
+        "//modules/common:macro",
+        "//modules/common/util:factory",
+        "//modules/prediction/predictor",
         "//modules/prediction/predictor/vehicle:free_move_predictor",
+        "//modules/prediction/predictor/vehicle:lane_sequence_predictor",
         "//modules/prediction/proto:prediction_conf_proto",
-        "//modules/common/util:factory",
-        "//modules/common:macro",
         "@glog//:glog",
-    ]
+    ],
 )
 
 cc_library(
@@ -22,27 +22,27 @@ cc_library(
     srcs = ["predictor_manager.cc"],
     hdrs = ["predictor_manager.h"],
     deps = [
-        "//modules/prediction/predictor:predictor",
-        "//modules/prediction/predictor:predictor_factory",
+        "//modules/common:macro",
+        "//modules/perception/proto:perception_proto",
         "//modules/prediction/container:container_manager",
         "//modules/prediction/container/obstacles:obstacles_container",
-        "//modules/perception/proto:perception_proto",
-        "//modules/prediction/proto:prediction_proto",
+        "//modules/prediction/predictor",
+        "//modules/prediction/predictor:predictor_factory",
         "//modules/prediction/proto:prediction_conf_proto",
-        "//modules/common:macro",
-        "@glog//:glog"
-    ]
+        "//modules/prediction/proto:prediction_proto",
+        "@glog//:glog",
+    ],
 )
 
 cc_library(
     name = "predictor",
-    hdrs = ["predictor.h"],
     srcs = ["predictor.cc"],
+    hdrs = ["predictor.h"],
     deps = [
-        "//modules/prediction/proto:prediction_proto",
-        "//modules/prediction/container/obstacles:obstacle",
         "//modules/common/proto:path_point_proto",
-    ]
+        "//modules/prediction/container/obstacles:obstacle",
+        "//modules/prediction/proto:prediction_proto",
+    ],
 )
 
-cpplint()
\ No newline at end of file
+cpplint()

modules/prediction/predictor/pedestrian/BUILD

@@ -4,16 +4,16 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "regional_predictor",
-    hdrs = ["regional_predictor.h"],
     srcs = ["regional_predictor.cc"],
+    hdrs = ["regional_predictor.h"],
     deps = [
-        "//modules/prediction/predictor:predictor",
-        "//modules/prediction/common:prediction_util",
-        "//modules/prediction/common:prediction_gflags",
-        "//modules/common/proto:path_point_proto",
         "//modules/common/math:kalman_filter",
+        "//modules/common/proto:path_point_proto",
+        "//modules/prediction/common:prediction_gflags",
+        "//modules/prediction/common:prediction_util",
+        "//modules/prediction/predictor",
         "@glog//:glog",
-    ]
+    ],
 )
 
-cpplint()
\ No newline at end of file
+cpplint()

modules/prediction/predictor/vehicle/BUILD

@@ -4,33 +4,33 @@ package(default_visibility = ["//visibility:public"])
 
 cc_library(
     name = "lane_sequence_predictor",
-    hdrs = ["lane_sequence_predictor.h"],
     srcs = ["lane_sequence_predictor.cc"],
+    hdrs = ["lane_sequence_predictor.h"],
     deps = [
-        "//modules/prediction/predictor:predictor",
+        "//modules/common:log",
+        "//modules/common/math:math_utils",
+        "//modules/common/proto:path_point_proto",
         "//modules/prediction/common:prediction_gflags",
         "//modules/prediction/common:prediction_map",
+        "//modules/prediction/predictor",
         "//modules/prediction/proto:lane_graph_proto",
-        "//modules/common/proto:path_point_proto",
-        "//modules/common:log",
-        "//modules/common/math:math_utils",
         "@eigen//:eigen",
-    ]
+    ],
 )
 
 cc_library(
     name = "free_move_predictor",
-    hdrs = ["free_move_predictor.h"],
     srcs = ["free_move_predictor.cc"],
+    hdrs = ["free_move_predictor.h"],
     deps = [
-        "//modules/prediction/predictor:predictor",
-        "//modules/prediction/proto:feature_proto",
-        "//modules/prediction/common:prediction_gflags",
-        "//modules/common/proto:path_point_proto",
-        "//modules/common/math:math_utils",
         "//modules/common:log",
+        "//modules/common/math:math_utils",
+        "//modules/common/proto:path_point_proto",
+        "//modules/prediction/common:prediction_gflags",
+        "//modules/prediction/predictor",
+        "//modules/prediction/proto:feature_proto",
         "@eigen//:eigen",
-    ]
+    ],
 )
 
-cpplint()
\ No newline at end of file
+cpplint()

modules/prediction/proto/BUILD

@@ -21,8 +21,8 @@ cc_proto_library(
     ],
     deps = [
         "//modules/common/proto:common_proto",
-        "//modules/perception/proto:perception_proto",
         "//modules/common/proto:path_point_proto",
+        "//modules/perception/proto:perception_proto",
     ],
 )