-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
kuka examples are broken #7314
Comments
I suspect it is due to #7166 and the fact that any directorPython you managed to load from
Will fix and add a test. |
As noted in RobotLocomotion#7314, the dual arm simulation of the iiwa described in kuka_sim_dual.pmd no longer works. This fixes it by conditonally ignoring the errors trying to find the frames to visualize, and adds a test to confirm that the simulation works.
Opened #7317 to fix the dual arm simulation. |
As noted in RobotLocomotion#7314, the dual arm simulation of the iiwa described in kuka_sim_dual.pmd no longer works. This fixes it by conditonally ignoring the errors trying to find the frames to visualize, and adds a test to confirm that the simulation works.
As noted in RobotLocomotion#7314, the dual arm simulation of the iiwa described in kuka_sim_dual.pmd no longer works. This fixes it by conditonally ignoring the errors trying to find the frames to visualize, and adds a test to confirm that the simulation works.
In light of the issue I just noted in #7024 (our prebuilt drake-visualizer can't load robot models), reviving director_ik_app.py is effectively impossible (which, uh, reduces the importance of the state translator). Given this state of affairs, I'm not totally sure what the best thing to do is. One idea would be to create a couple of canned demos which send a canned plan to a simulation (probably single arm and dual arm versions) either by doing some IK or just recording the series of joint states and sending a trajectory. Does that sound worthwhile? Other ideas? |
if the pick and place demos are coming soon from @avalenzu , then perhaps they can play that role? N.B. - @sammy-tri i think you might have linked to the wrong issue number above? |
This came up in RobotLocomotion/drake-iiwa-driver#14 (comment) -- without a working directorPython built against drake, providing a graphical IK example is going to be tough. This may call for increasing the priority of #7024 (built against a fixed drake target) or finding a way to package directorPython along with the files currently in |
Just to check, are these issues still relevant? This is currently marked as high-priority, would be nice to either lower the priority or close the issue if possible. |
I think it's up to Russ to determine if the manipulation station examples are sufficient to close this ticket (I'm leaning toward "yes" on that question). I'll lower the priority at any rate. |
I guess I should elaborate on my proposal for closing, since I don't think I mean just pressing the "close" button: Resolve this and #7321 by removing all director-based examples and any other broken stuff. |
These haven't worked in a long time since we haven't had a version of director with the needed functionality available. Fixes RobotLocomotion#7314 Fixes RobotLocomotion#7321
After partial repairs in #7313, I still see the following state of affairs in my kuka examples directory:
How do we put better test coverage on these?
The text was updated successfully, but these errors were encountered: