wip: add dynamic yaw limits to auto aim and remove unused package fro… - #66
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (8)
💤 Files with no reviewable changes (2)
Walkthrough本次改动新增 Changes动态 yaw 速度/加速度约束
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant Gimbal
participant AutoAimComponent
participant AutoAimContext as AutoAim::Context
participant AutoAimLoop as AutoAim主循环
participant FireControllerV2
Gimbal->>AutoAimComponent: yaw_velocity(rad/s)
AutoAimComponent->>AutoAimComponent: 差分估算max_yaw_acc/max_yaw_vel
AutoAimComponent->>AutoAimContext: ctx.max_yaw_vel / ctx.max_yaw_acc
AutoAimLoop->>AutoAimContext: 读取max_yaw_vel/max_yaw_acc
AutoAimLoop->>FireControllerV2: update(State{timestamp,yaw,pitch,max_yaw_vel,max_yaw_acc})
FireControllerV2->>FireControllerV2: get_attack_window()使用动态限值
Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…m cmake
本次修改为自瞄链路加入了动态 yaw 限制,并同步调整了火控接口与配置:
/auto_aim/yaw_velocity输入,AutoAimComponent会根据 yaw 角速度估算并传递max_yaw_vel、max_yaw_acc。AutoAim::Context增加 yaw 相关动态约束字段,主循环与弹道/开火控制流程改为使用这些运行时限制。FireControllerV2重构为通过State结构传入时间、姿态和动态约束,update接口随之调整。max_acc、max_vel以及 TinyMPC 相关 CMake 拉取逻辑。