통합 프로토타입 업데이트 - #12
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| from rclpy.executors import ExternalShutdownException | ||
| from rclpy.node import Node | ||
| from sensor_msgs.msg import Image | ||
| from ultralytics import YOLO |
There was a problem hiding this comment.
Declare the YOLO runtime dependency
When this node is launched in an environment provisioned from the package metadata, the new ultralytics import is not installed or tracked by package.xml alongside the other Python/ROS runtime dependencies. That makes ros2 launch perception_pkg cctv_perception.launch.py fail immediately with ModuleNotFoundError: ultralytics unless the operator has manually installed an undeclared dependency, so the package should declare/provision this runtime requirement explicitly.
Useful? React with 👍 / 👎.
* 3번 노드 기능 + 5번 노드 기능 + bringup 폴더 추가 * 통합 프로토타입 * 통합 프로토타입 업데이트
Summary
Notes
mainwas rejected by repository rules, so this PR routes the update through the required workflow.