rdelfin / min-review-bot

MIT License
2 stars 0 forks source link

Bug in owner calculation #1

Open rdelfin opened 1 year ago

rdelfin commented 1 year ago

Find a sample incorrect owner calculation below. Note that this should require both robot-owners and gen2-owners, but currently has them as either-or only

File Owners

The minimum set of reviewers required are: (@wayveai/rust-reviewers && @wayveai/secdevops-owners && @wayveai/data-explorers-owners && (@wayveai/robot-owners || @wayveai/gen2-owners) && @wayveai/robot-web && (@wayveai/deep-robotics || @wayveai/robot-owners || @wayveai/robot-software))

Details
  • 3rdparty/rust/Cargo.Bazel.lock
    • @wayveai/rust-reviewers
  • 3rdparty/rust/cxxbridge-cmd.Cargo.Bazel.lock
    • @wayveai/rust-reviewers
  • WORKSPACE
    • @wayveai/secdevops-owners
  • examples/rust/tonic/server.rs
    • @wayveai/secdevops-owners
  • wayve/core/data/vehicle_calibration.rs
    • @wayveai/data-explorers-owners
  • wayve/robot/3rdparty/rules_r2r.patch
    • @wayveai/robot-owners
    • @wayveai/gen2-owners
  • wayve/robot/gen2/audio_recorder/src/gstreamer_pipeline.rs
    • @wayveai/gen2-owners
  • wayve/robot/gen2/log_composer/src/mcap_tools/utils.rs
    • @wayveai/gen2-owners
  • wayve/robot/modules/radar/driver/examples/radar_demo.rs
    • @wayveai/gen2-owners
  • wayve/robot/nodes/grpc_agent_node/test_utils/ros.rs
    • @wayveai/robot-web
  • wayve/robot/telemetry_agent/test_utils/src/utils.rs
    • @wayveai/robot-web
  • wayve/robot/telemetry_agent/tests/subscriber/forwarder_test.rs
    • @wayveai/robot-web
  • wayve/robot/tools/sample_data_generator/src/video.rs
    • @wayveai/deep-robotics
    • @wayveai/robot-owners
    • @wayveai/robot-software
  • wayve/services/fleet360/src/etcd.rs
    • @wayveai/robot-web
  • wayve/services/fleet360/src/onelogin/client.rs
    • @wayveai/robot-web
  • wayve/services/fleet360/src/telemetry.rs
    • @wayveai/robot-web
  • wayve/services/fleet360/src/test_utils/kafka.rs
    • @wayveai/robot-web
  • wayve/services/fleet360/src/test_utils/mod.rs
    • @wayveai/robot-web
jltorresm commented 2 months ago

Find another example here:

The min-review-bot set is missing wayveai/neural-fleet-owners, required by github

image

File Owners

The minimum set of reviewers required are: ((@wayveai/all-sim-group-owners || @wayveai/graph-sim-owners) && @wayveai/ai-owners && @wayveai/ai-lib-owners && @wayveai/sim-infra-owners && @wayveai/data-explorers-owners && (@wayveai/core-cpp-owners || @wayveai/perception-owners) && @wayveai/ui-owners)

Details
  • wayve/ai/evaluation/crc/autopublish.yaml
    • @wayveai/all-sim-group-owners
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/crc/graph_nodes/runner.py
    • @wayveai/all-sim-group-owners
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/ghost_gym/graph_sim/autopublish.yaml
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/ghost_gym/graph_sim/nodes/runner.py
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/image_quality_gym/autopublish.yaml
    • @wayveai/neural-fleet-owners
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/image_quality_gym/nodes/runner.py
    • @wayveai/neural-fleet-owners
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/minkowski_gym/nodes/runner_config.py
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/robot_repro_gym/nodes/runner_config.py
    • @wayveai/ai-owners
  • wayve/ai/evaluation/shadow_gym/new_backend/shadow_gym_nodes.py
    • @wayveai/graph-sim-owners
  • wayve/ai/evaluation/smooth_gym/nodes/runner_config.py
    • @wayveai/ai-owners
  • wayve/ai/lib/interfaces_v2.py
    • @wayveai/ai-lib-owners
  • wayve/ai/nvs/services/ghost_gym/autopublish.yaml
    • @wayveai/neural-fleet-owners
  • wayve/ai/nvs/services/workflow/autopublish.yaml
    • @wayveai/neural-fleet-owners
  • wayve/ai/robot_replay/robot_replay.py
    • @wayveai/sim-infra-owners
  • wayve/ai/services/ghost_gym/api/autopublish.yaml
    • @wayveai/sim-infra-owners
  • wayve/ai/services/ghost_gym/workers/cpu/autopublish.yaml
    • @wayveai/sim-infra-owners
  • wayve/ai/services/ghost_gym/workers/evaluate/autopublish.yaml
    • @wayveai/sim-infra-owners
  • wayve/ai/services/shadow_gym/api/autopublish.yaml
    • @wayveai/sim-infra-owners
  • wayve/ai/services/shadow_gym/worker/autopublish.yaml
    • @wayveai/sim-infra-owners
  • wayve/ai/sim/robot/simulated_inference_module.py
    • @wayveai/all-sim-group-owners
  • wayve/core/cpp/include/wayve/core/json_tools.hpp
    • @wayveai/core-cpp-owners
    • @wayveai/all-sim-group-owners
  • wayve/core/data/BUILD
    • @wayveai/data-explorers-owners
  • wayve/core/data/data_utils.py
    • @wayveai/data-explorers-owners
  • wayve/core/imaging/pybind/image_loader_pybindings.cpp
    • @wayveai/core-cpp-owners
    • @wayveai/perception-owners
    • @wayveai/total-robot-software-platform
  • wayve/frontends/console/autopublish.yaml
    • @wayveai/ui-owners
  • wayve/frontends/generated/pb/wayve/interfaces/protobuf/model_inputs.ts
  • wayve/frontends/generated/pb/wayve/interfaces/protobuf/model_outputs.ts
  • wayve/frontends/hil-config-ui/autopublish.yaml
    • @wayveai/ui-owners
  • wayve/frontends/robot-ui/autopublish.yaml
    • @wayveai/robot-web
    • @wayveai/ui-owners
  • wayve/interfaces/protobuf/model_inputs.proto
    • @wayveai/data-explorers-owners
    • @wayveai/robot-owners
    • @wayveai/gen2
  • wayve/interfaces/protobuf/model_outputs.proto
    • @wayveai/data-explorers-owners
    • @wayveai/robot-owners
    • @wayveai/gen2
  • wayve/robot/coordinator/BUILD
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/autopublish.yaml
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/services/cortex.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/services/cortex_subconfigs.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/tasks/autonomous_driving.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/tasks/local_model_validation.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/test/data/cortex_autonomous_driving_config.json
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/coordinator/test/test_configs.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/cortex/src/lib/inference_output_graph.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/inference/include/wayve/robot/modules/inference/model_output_v2.hpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/BUILD
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/include/wayve/robot/modules/output_processor/classification_processor.hpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/include/wayve/robot/modules/output_processor/debug_tensor_processor.hpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/include/wayve/robot/modules/output_processor/driving_plan_processor_v2.hpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/include/wayve/robot/modules/output_processor/driving_plan_processor_v2_wrapper.hpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/include/wayve/robot/modules/output_processor/output_processor_config.hpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/src/classification_processor.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/src/debug_tensor_processor.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/src/driving_plan_processor_v2.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/src/driving_plan_processor_v2_wrapper.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/src/output_processor_config.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/src/pybind_driving_plan_processor.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/test/test_driving_plan_processor_v2.cpp
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/test/test_pybind_driving_plan_processor.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/modules/output_processor/test/test_pybind_output_processor.py
    • @wayveai/robot-owners
    • @wayveai/gen1
    • @wayveai/total-robot-software-platform
  • wayve/robot/tools/mender_release_assistant/autopublish.yaml
    • @wayveai/robot-owners
    • @wayveai/robot-software
    • @wayveai/total-robot-software-platform
  • wayve/sim/repro/BUILD
    • @wayveai/all-sim-group-owners
  • wayve/sim/repro/graph_nodes/examples/run_example.py
    • @wayveai/all-sim-group-owners
  • wayve/sim/repro/graph_nodes/model_info.py
    • @wayveai/all-sim-group-owners
  • wayve/sim/repro/include/wayve/sim/repro/runner_config.hpp
    • @wayveai/all-sim-group-owners
    • @wayveai/core-cpp-owners
  • wayve/sim/repro/run.py
    • @wayveai/all-sim-group-owners
  • wayve/sim/repro/src/runner.cpp
    • @wayveai/all-sim-group-owners
    • @wayveai/core-cpp-owners
  • wayve/sim/repro/src/runner_config_pybind.cpp
    • @wayveai/all-sim-group-owners
    • @wayveai/core-cpp-owners
  • wayve/sim/repro/test/test_runner.py
    • @wayveai/all-sim-group-owners