uci-uav-forge / uavf_2024

MIT License
2 stars 0 forks source link

Refactor ImagingNode to a Normal Class #226

Closed MinhxNguyen7 closed 2 weeks ago

MinhxNguyen7 commented 3 weeks ago

Summary

Test Plan

Issues

MinhxNguyen7 commented 3 weeks ago

@Dat-Bois This is why the PoseProvider exists. Abstracting the subscription data collection meant that, with dependency injection, I can move the functionality of ImagingNode to another class, and this new class doesn't even need to know that ROS exists. In the future, we can switch out MAVROS for MAVSDK in the implementation of PoseProvider, and the perception module wouldn't have to change at all.