DAIRLab / dairlib

MIT License
65 stars 26 forks source link

Update RosSubscriberSystem #310

Closed mposa closed 1 year ago

mposa commented 1 year ago

Update RosSubscriberSystem to declare abstract state, rather than set via AllocateAbstractState.

The previous implementation was based on LcmSubscriberSystem, which was updated in Drake https://github.com/RobotLocomotion/drake/commit/10d901256ba286892ba813dc13e73b6cfa48f56a, and now causes ./systems/ros/test_ros_subscriber_system to segfault (caught by @adamw8 )

This PR mirrors some of the upstream changes, specifically how it handles the abstract state declaration and initialization.


This change is Reviewable