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.

Sign up for GitHub

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

Jump to bottom

Re-enable PILZ runtime tests #833

Open
henningkayser opened this issue Nov 29, 2021 · 2 comments
Open

Re-enable PILZ runtime tests #833

henningkayser opened this issue Nov 29, 2021 · 2 comments
Assignees
@Abishalini
Labels
enhancement New feature or request

Comments

@henningkayser
Copy link
Member

Related to: moveit/moveit#2947
The migration of the planner is completed and all the tests pass with the PRBT (#315).
However, we don't want to get the PRBT robot merged into moveit_resources, instead we should fix and enable all tests to use some other robot that still gets maintained.
@henningkayser henningkayser added the enhancement New feature or request label Nov 29, 2021
@henningkayser henningkayser self-assigned this Nov 29, 2021
@vatanaksoytezer
Copy link
Contributor

As far as I know the tests are enabled now, can we close this @henningkayser ?

@henningkayser
Copy link
Member Author

@vatanaksoytezer not really, only the unit tests are enabled by now. The integration tests are still disabled. @Abishalini is starting to work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Abishalini Abishalini

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@henningkayser @vatanaksoytezer @Abishalini