Skip to content

Commit

Permalink
Temporarily disable tree-based AG type selection
Browse files Browse the repository at this point in the history
  • Loading branch information
cgokmen committed Dec 13, 2023
1 parent 00ce7c5 commit 4b6ecc1
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions omnigibson/robots/manipulation_robot.py
Original file line number Diff line number Diff line change
Expand Up @@ -1057,12 +1057,13 @@ def _get_assisted_grasp_joint_type(self, ag_obj, ag_link):
# Otherwise, compute the joint type. We use a fixed joint unless the link is a non-fixed link.
# A link is non-fixed if it has any non-fixed parent joints.
joint_type = "FixedJoint"
articulation_graph = ag_obj.articulation_graph
for edge in nx.edge_dfs(articulation_graph, ag_link, orientation="reverse"):
joint = articulation_graph.edges[edge]["joint"]
if joint.joint_type != JointType.JOINT_FIXED:
joint_type = "SphericalJoint"
break
# TODO: Add this logic back w/o the graph using just the articulationview.
# articulation_graph = ag_obj.articulation_graph
# for edge in nx.edge_dfs(articulation_graph, ag_link, orientation="reverse"):
# joint = articulation_graph.edges[edge]["joint"]
# if joint.joint_type != JointType.JOINT_FIXED:
# joint_type = "SphericalJoint"
# break

return joint_type

Expand Down

0 comments on commit 4b6ecc1

Please sign in to comment.