Skip to content

Commit cd0e541

Browse files
rjgriffin42stefanfasanoAlexander OCU
authored
Feature/separate csg values (#1103)
* changed csg status to publish current velocity commands in velocity units * made dedicated class for ROS2 step generator communication help. integrated that into CSG xbox and CSG rdx panel * added setup for different forward/backwards max step lengths * made it so xbox and rdx csg panel don't publish by default all the time. also made velocity commands in csg panel in m/s * fixed some bugs with csg panel * added swing height control to rdx csg panel * added a separate set of stepping parameters for step generation vs control * did some cleanup of the stepping parameters calls * switched some of the parameters --------- Co-authored-by: Stefan Fasano <sfasano@ihmc.org> Co-authored-by: Alexander OCU <rosie@ihmc.org>
1 parent f5640a0 commit cd0e541

File tree

30 files changed

+132
-962
lines changed

30 files changed

+132
-962
lines changed

ihmc-avatar-interfaces/src/main/java/us/ihmc/avatar/joystickBasedJavaFXController/ContinuousStepController.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -81,9 +81,13 @@ public class ContinuousStepController
8181

8282
public ContinuousStepController(WalkingControllerParameters walkingControllerParameters)
8383
{
84-
steppingParameters = walkingControllerParameters.getSteppingParameters();
84+
steppingParameters = walkingControllerParameters.getSteppingParametersForStepGeneration();
8585

86-
snapAndWiggleParameters.setFootLength(walkingControllerParameters.getSteppingParameters().getFootLength());
86+
double footLength = steppingParameters.getFootLength();
87+
double toeWidth = steppingParameters.getToeWidth();
88+
double footWidth = steppingParameters.getFootWidth();
89+
90+
snapAndWiggleParameters.setFootLength(footLength);
8791
snapAndWiggleSingleStep = new SnapAndWiggleSingleStep(snapAndWiggleParameters);
8892

8993
continuousStepGenerator.setNumberOfTicksBeforeSubmittingFootsteps(0);
@@ -124,10 +128,7 @@ public boolean isUnitVelocity()
124128
continuousStepGenerator.addFootstepValidityIndicator(this::isSafeDistanceFromObstacle);
125129
continuousStepGenerator.addFootstepValidityIndicator(this::isSafeStepHeight);
126130

127-
SteppingParameters steppingParameters = walkingControllerParameters.getSteppingParameters();
128-
double footLength = steppingParameters.getFootLength();
129-
double toeWidth = steppingParameters.getToeWidth();
130-
double footWidth = steppingParameters.getFootWidth();
131+
131132
ConvexPolygon2D footPolygon = new ConvexPolygon2D();
132133
footPolygon.addVertex(footLength / 2.0, toeWidth / 2.0);
133134
footPolygon.addVertex(footLength / 2.0, -toeWidth / 2.0);

ihmc-avatar-interfaces/src/main/java/us/ihmc/avatar/joystickBasedJavaFXController/JoystickStepParametersProperty.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ public void set(WalkingControllerParameters walkingControllerParameters)
121121
swingDuration = walkingControllerParameters.getDefaultSwingTime();
122122
transferDuration = walkingControllerParameters.getDefaultTransferTime();
123123

124-
SteppingParameters steppingParameters = walkingControllerParameters.getSteppingParameters();
124+
SteppingParameters steppingParameters = walkingControllerParameters.getSteppingParametersForStepGeneration();
125125
swingHeight = walkingControllerParameters.getSwingTrajectoryParameters().getMinSwingHeight();
126126
maxStepLength = steppingParameters.getMaxStepLength();
127127
defaultStepWidth = steppingParameters.getInPlaceWidth();

ihmc-avatar-interfaces/src/main/java/us/ihmc/avatar/joystickBasedJavaFXController/XBoxOneCSGPlugin.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ private void configureCSGParameters(ContinuousStepGeneratorParametersMessage csg
205205
csgParametersCommand.setSwingDuration(walkingControllerParameters.getDefaultSwingTime());
206206
csgParametersCommand.setTransferDuration(walkingControllerParameters.getDefaultTransferTime());
207207

208-
SteppingParameters steppingParameters = walkingControllerParameters.getSteppingParameters();
208+
SteppingParameters steppingParameters = walkingControllerParameters.getSteppingParametersForStepGeneration();
209209
csgParametersCommand.setSwingHeight(walkingControllerParameters.getSwingTrajectoryParameters().getDefaultSwingHeight());
210210
csgParametersCommand.setMaxStepLengthForwards(steppingParameters.getMaxStepLength());
211211
csgParametersCommand.setMaxStepLengthBackwards(steppingParameters.getMaxBackwardStepLength());

ihmc-avatar-interfaces/src/main/java/us/ihmc/avatar/networkProcessor/directionalControlToolboxModule/DirectionalControlController.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
import us.ihmc.euclid.geometry.interfaces.ConvexPolygon2DReadOnly;
3030
import us.ihmc.euclid.referenceFrame.FramePose3D;
3131
import us.ihmc.euclid.referenceFrame.ReferenceFrame;
32-
import us.ihmc.euclid.referenceFrame.interfaces.FixedFramePose3DBasics;
3332
import us.ihmc.euclid.referenceFrame.interfaces.FramePose2DReadOnly;
3433
import us.ihmc.euclid.referenceFrame.interfaces.FramePose3DReadOnly;
3534
import us.ihmc.euclid.transform.RigidBodyTransform;
@@ -198,7 +197,7 @@ public DirectionalControlController(FullHumanoidRobotModel robotModel,
198197
this.footPolygons = getFootPolygons(walkingControllerParameters);
199198
this.robotUpdater = new RobotModelUpdater(robotModel);
200199

201-
steppingParameters = walkingControllerParameters.getSteppingParameters();
200+
steppingParameters = walkingControllerParameters.getSteppingParametersForStepGeneration();
202201
stepTime = () -> getSwingDuration() + getTransferDuration();
203202

204203
controlParameters = new JoystickStepParameters(walkingControllerParameters);
@@ -210,7 +209,7 @@ public DirectionalControlController(FullHumanoidRobotModel robotModel,
210209
controlParameters = userProfileManager.loadProfile("default");
211210
stepParametersReference = new AtomicReference<JoystickStepParameters>(controlParameters);
212211

213-
snapAndWiggleParameters.setFootLength(walkingControllerParameters.getSteppingParameters().getFootLength());
212+
snapAndWiggleParameters.setFootLength(steppingParameters.getFootLength());
214213
snapAndWiggleSingleStep = new SnapAndWiggleSingleStep(snapAndWiggleParameters);
215214

216215
continuousStepGenerator.setNumberOfFootstepsToPlan(10);
@@ -399,7 +398,6 @@ public void run()
399398
*/
400399
private SideDependentList<ConvexPolygon2D> getFootPolygons(WalkingControllerParameters walkingControllerParameters)
401400
{
402-
SteppingParameters steppingParameters = walkingControllerParameters.getSteppingParameters();
403401
double footLength = steppingParameters.getFootLength();
404402
double footWidth = steppingParameters.getFootWidth();
405403
ConvexPolygon2D footPolygon = new ConvexPolygon2D();

ihmc-avatar-interfaces/src/main/java/us/ihmc/avatar/networkProcessor/stepConstraintToolboxModule/StepConstraintToolboxController.java

Lines changed: 0 additions & 209 deletions
This file was deleted.

ihmc-avatar-interfaces/src/main/java/us/ihmc/avatar/scs2/SCS2AvatarSimulationFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -570,7 +570,7 @@ private void setupStepGeneratorThread()
570570
.getContactPointParameters(),
571571
robotModel.get()
572572
.getWalkingControllerParameters()
573-
.getSteppingParameters());
573+
.getSteppingParametersForStepGeneration());
574574
stepSnapperUpdatable.setSnapToHeightMap(true);
575575
}
576576

0 commit comments

Comments
 (0)