Skip to content
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 a new unit test for clone env leakage. #288

Open
psigen opened this issue Apr 1, 2016 · 1 comment
Open

Add a new unit test for clone env leakage. #288

psigen opened this issue Apr 1, 2016 · 1 comment

Comments

@psigen
Copy link
Member

psigen commented Apr 1, 2016

After the discovery of #286, we should add a unit test that verifies that cloning and planning operations do not leak OpenRAVE environments. This would be something to the effect of:

import openravepy

robot.PlanToConfiguration(robot.GetActiveDOFValues(), execute=True)
initial_num_envs = len(openravepy.RaveGetEnvironments())
for i in range(10):
    robot.PlanToConfiguration(robot.GetActiveDOFValues(), execute=True)
assertEqual(initial_num_envs, openravepy.RaveGetEnvironments())

I am not sure exactly how to do this test in prpy directly (without herbpy or similar).

@mkoval
Copy link
Member

mkoval commented Apr 1, 2016

It may be best to put this code in each *py package. It's possible for a robot to introduce circular dependencies in its CloneBindings functions, so that is the most reliably place to do the test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants