Skip to content

Commit 674891c

Browse files
committed
Delete ReferenceFrameLibrary.
1 parent 4059fcf commit 674891c

34 files changed

+143
-242
lines changed

ihmc-communication/src/main/java/us/ihmc/communication/crdt/CRDTDetachableReferenceFrame.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,22 +2,23 @@
22

33
import us.ihmc.euclid.referenceFrame.ReferenceFrame;
44
import us.ihmc.robotics.referenceFrames.DetachableReferenceFrame;
5-
import us.ihmc.robotics.referenceFrames.ReferenceFrameLibrary;
5+
6+
import java.util.function.Function;
67

78
public class CRDTDetachableReferenceFrame
89
{
910
private final CRDTBidirectionalString parentFrameName;
1011
private final CRDTBidirectionalRigidBodyTransform transformToParent;
1112
private final DetachableReferenceFrame detachableReferenceFrame;
1213

13-
public CRDTDetachableReferenceFrame(ReferenceFrameLibrary referenceFrameLibrary,
14+
public CRDTDetachableReferenceFrame(Function<String, ReferenceFrame> frameFunction,
1415
CRDTBidirectionalString parentFrameName,
1516
CRDTBidirectionalRigidBodyTransform transformToParent)
1617
{
1718
this.parentFrameName = parentFrameName;
1819
this.transformToParent = transformToParent;
1920

20-
detachableReferenceFrame = new DetachableReferenceFrame(referenceFrameLibrary, transformToParent.getValueReadOnly());
21+
detachableReferenceFrame = new DetachableReferenceFrame(frameFunction, transformToParent.getValueReadOnly());
2122
}
2223

2324
public void update()

ihmc-graphics/src/libgdx/java/us/ihmc/rdx/imgui/ImGuiReferenceFrameLibraryCombo.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import imgui.ImGui;
44
import imgui.flag.ImGuiCol;
55
import org.apache.commons.lang3.ArrayUtils;
6-
import us.ihmc.robotics.referenceFrames.ReferenceFrameLibrary;
76

87
import java.util.SortedSet;
98
import java.util.TreeSet;
@@ -14,12 +13,14 @@
1413
* Used to select between the reference frames in a library by human readable names.
1514
* It also includes any values that the user already might have and keeps those around
1615
* even if deselected. Also, this is done immediate mode style.
16+
*
17+
* TODO: Fix up with new design. Probably rename to be less general, like RDXBehaviorSceneFrameSelector...
1718
*/
1819
public class ImGuiReferenceFrameLibraryCombo
1920
{
2021
private final ImGuiUniqueLabelMap labels = new ImGuiUniqueLabelMap(getClass());
2122
private final String comboName;
22-
private final ReferenceFrameLibrary referenceFrameLibrary;
23+
private final Consumer<Consumer<String>> frameSupplier;
2324
private final Supplier<String> currentFrameNameGetter;
2425
private final Consumer<String> currentFrameNameSetter;
2526
private final SortedSet<String> referenceFrameLibraryNames = new TreeSet<>();
@@ -29,20 +30,20 @@ public class ImGuiReferenceFrameLibraryCombo
2930
private transient String[] selectableReferenceFrameNameArray = new String[0];
3031

3132
public ImGuiReferenceFrameLibraryCombo(String comboName,
32-
ReferenceFrameLibrary referenceFrameLibrary,
33+
Consumer<Consumer<String>> frameSupplier,
3334
Supplier<String> currentFrameNameGetter,
3435
Consumer<String> currentFrameNameSetter)
3536
{
3637
this.comboName = comboName;
37-
this.referenceFrameLibrary = referenceFrameLibrary;
38+
this.frameSupplier = frameSupplier;
3839
this.currentFrameNameGetter = currentFrameNameGetter;
3940
this.currentFrameNameSetter = currentFrameNameSetter;
4041
}
4142

4243
public void render()
4344
{
4445
referenceFrameLibraryNames.clear();
45-
referenceFrameLibrary.getAllFrameNames(referenceFrameLibraryNames::add);
46+
frameSupplier.accept(referenceFrameLibraryNames::add);
4647

4748
selectableReferenceFrameNames.add(currentFrameNameGetter.get());
4849
selectableReferenceFrameNames.addAll(referenceFrameLibraryNames);

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/RDXAvailableBehaviorTreeFile.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public void update()
3636
referenceFramesInWorld.clear();
3737
for (String referenceFrameName : referenceFrameNames)
3838
{
39-
ReferenceFrame frameByName = scene.getReferenceFrameLibrary().findFrameByName(referenceFrameName);
39+
ReferenceFrame frameByName = scene.findFrameByName(referenceFrameName);
4040
if (frameByName != null && frameByName.getRootFrame() == ReferenceFrame.getWorldFrame())
4141
{
4242
referenceFramesInWorld.add(referenceFrameName);

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/RDXBehaviorTree.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,16 @@ public class RDXBehaviorTree extends BehaviorTree<RDXBehaviorTreeRootNode, RDXBe
4646
private boolean draggingDivider;
4747
private boolean shouldSave = false;
4848

49-
public RDXBehaviorTree(WorkspaceResourceDirectory treeFilesDirectory,ROS2SyncedRobotModel syncedRobot,
49+
public RDXBehaviorTree(WorkspaceResourceDirectory treeFilesDirectory,
50+
ROS2SyncedRobotModel syncedRobot,
5051
ROS2PeerClockOffsetEstimator peerClockEstimator,
5152
RobotCollisionModel selectionCollisionModel,
5253
RDXBaseUI baseUI,
5354
RDX3DPanel panel3D)
5455
{
5556
super(syncedRobot, ROS2ActorDesignation.OPERATOR, peerClockEstimator, treeFilesDirectory, new RDXBehaviorTreeNodeBuilder());
5657

57-
scene = new RDXBehaviorTreeScene(panel);
58+
scene = new RDXBehaviorTreeScene(syncedRobot, panel);
5859

5960
((RDXBehaviorTreeNodeBuilder) getNodeBuilder()).initialize(crdtInfo,
6061
saveFileDirectory,

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/RDXBehaviorTreeNode.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
import us.ihmc.avatar.drcRobot.ROS2SyncedRobotModel;
1414
import us.ihmc.behaviors.behaviorTree.*;
1515
import us.ihmc.behaviors.behaviorTree.log.BehaviorTreeNodeMessageLogger.LogMessage;
16+
import us.ihmc.rdx.behaviorTree.scene.RDXBehaviorTreeScene;
1617
import us.ihmc.rdx.imgui.ImGuiTools;
1718
import us.ihmc.rdx.imgui.ImGuiUniqueLabelMap;
1819
import us.ihmc.rdx.input.ImGui3DViewInput;
@@ -21,7 +22,6 @@
2122
import us.ihmc.rdx.ui.tools.ImGuiScrollableLogArea;
2223
import us.ihmc.rdx.vr.RDXVRContext;
2324
import us.ihmc.robotics.physics.RobotCollisionModel;
24-
import us.ihmc.robotics.referenceFrames.ReferenceFrameLibrary;
2525

2626
import javax.annotation.Nullable;
2727
import java.util.ArrayList;
@@ -67,7 +67,7 @@ public class RDXBehaviorTreeNode<S extends BehaviorTreeNodeState<D>,
6767
private boolean dragReleasedAfter = false;
6868

6969
protected final DRCRobotModel robotModel;
70-
protected final ReferenceFrameLibrary referenceFrameLibrary;
70+
protected final RDXBehaviorTreeScene scene;
7171
protected final ROS2SyncedRobotModel syncedRobot;
7272
protected final RobotCollisionModel selectionCollisionModel;
7373
protected final RDXBaseUI baseUI;
@@ -86,7 +86,7 @@ public RDXBehaviorTreeNode(S state, RDXBehaviorTreeRootNode rootNode)
8686
this.state = state;
8787
this.rootNode = rootNode;
8888
this.robotModel = rootNode.getDefinition().getRobotModel();
89-
this.referenceFrameLibrary = rootNode.getState().getScene().getReferenceFrameLibrary();
89+
this.scene = rootNode.getScene();
9090
this.syncedRobot = rootNode.getSyncedRobot();
9191
this.selectionCollisionModel = rootNode.getSelectionCollisionModel();
9292
this.baseUI = rootNode.getBaseUI();
@@ -104,7 +104,7 @@ public RDXBehaviorTreeNode(S state,
104104
this.state = state;
105105
this.rootNode = (RDXBehaviorTreeRootNode) this;
106106
this.robotModel = rootNode.getDefinition().getRobotModel();
107-
this.referenceFrameLibrary = rootNode.getState().getScene().getReferenceFrameLibrary();
107+
this.scene = rootNode.getScene();
108108
this.syncedRobot = syncedRobot;
109109
this.selectionCollisionModel = selectionCollisionModel;
110110
this.baseUI = baseUI;

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/RDXBehaviorTreeRootNode.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -235,6 +235,11 @@ public ROS2SyncedRobotModel getSyncedRobot()
235235
return syncedRobot;
236236
}
237237

238+
public RDXBehaviorTreeScene getScene()
239+
{
240+
return scene;
241+
}
242+
238243
public RobotCollisionModel getSelectionCollisionModel()
239244
{
240245
return selectionCollisionModel;

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/actions/RDXChestOrientationAction.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public RDXChestOrientationAction(long id, RDXBehaviorTreeRootNode rootNode)
6161
definition::setHoldPoseInWorldLater,
6262
imBoolean -> ImGui.checkbox(labels.get("Hold pose in world later"), imBoolean));
6363
parentFrameComboBox = new ImGuiReferenceFrameLibraryCombo("Parent frame",
64-
referenceFrameLibrary,
64+
scene::getAllFrameNames,
6565
definition::getParentFrameName,
6666
state.getChestFrame()::changeFrame);
6767
yawWidget = new ImDoubleWrapper(definition.getRotationReadOnly()::getYaw, definition::setYaw,

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/actions/RDXFootPoseAction.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ public RDXFootPoseAction(long id, RDXBehaviorTreeRootNode rootNode)
5454
poseGizmo.create(panel3D);
5555

5656
parentFrameComboBox = new ImGuiReferenceFrameLibraryCombo("Parent frame",
57-
referenceFrameLibrary,
57+
scene::getAllFrameNames,
5858
definition::getParentFrameName,
5959
state.getFootFrame()::changeFrame);
6060
trajectoryDurationWidget = new ImDoubleWrapper(definition::getTrajectoryDuration,

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/actions/RDXFootstepPlanAction.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public RDXFootstepPlanAction(long id, RDXBehaviorTreeRootNode rootNode)
7878
RecyclingArrayListTools.getUnsafe(state.getManuallyPlacedFootsteps(), numberOfAllocatedFootsteps++)));
7979

8080
parentFrameComboBox = new ImGuiReferenceFrameLibraryCombo("Parent frame",
81-
referenceFrameLibrary,
81+
scene::getAllFrameNames,
8282
definition::getParentFrameName,
8383
this::changeParentFrame);
8484
manuallyPlaceStepsWrapper = new ImBooleanWrapper(definition::getIsManuallyPlaced,

ihmc-high-level-behaviors/src/libgdx/java/us/ihmc/rdx/behaviorTree/actions/RDXHandPoseAction.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ public RDXHandPoseAction(long id, RDXBehaviorTreeRootNode rootNode)
228228
}
229229

230230
parentFrameComboBox = new ImGuiReferenceFrameLibraryCombo("Parent frame",
231-
referenceFrameLibrary,
231+
scene::getAllFrameNames,
232232
definition::getPalmParentFrameName,
233233
state.getPalmFrame()::changeFrame);
234234

0 commit comments

Comments
 (0)