-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Hiking to the Perception and Autonomy Process and Behavior UI #228
base: develop
Are you sure you want to change the base?
Conversation
@@ -97,7 +99,23 @@ public RDXContinuousHikingPanel(ROS2Helper ros2Helper, | |||
this.activePerceptionModule = activePerceptionModule; | |||
this.swingPlannerParameters = swingPlannerParameters; | |||
this.swingTrajectoryParameters = swingTrajectoryParameters; | |||
this.continuousHikingParameters = continuousHikingParameters; | |||
|
|||
if (continuousHikingParameters == null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PotatoPeeler3000 This is the place I need help with. In your setup you create a ImGuiRemoteROS2StoredPropertySetGroup in the perception UI, but we're not using that according to our design and therefore the parameters don't get synced with just RDXStoredPropertySetTuner. So I'm trying to strike a balance of not breaking your stuff while being able to proceed with the newer design.
import us.ihmc.tools.thread.RestartableThrottledThread; | ||
|
||
/** | ||
* This class takes care of stuff to run continuous hiking in the {@link us.ihmc.PerceptionAndAutonomyProcess}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this class is specific to PerceptionAndAutonomyProcess, it should go in the same package as it.
… is working, fixed synced robot
No description provided.