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

Feature/tsr gripper library #313

Merged
merged 7 commits into from
Jul 7, 2016
Merged

Feature/tsr gripper library #313

merged 7 commits into from
Jul 7, 2016

Conversation

athackst
Copy link

@athackst athackst commented Jul 2, 2016

adding support for generic robots

@athackst
Copy link
Author

athackst commented Jul 6, 2016

@mkoval @jeking04 I don't think this will have any adverse affects to current prpy operation, so we could go ahead and merge it in.

@@ -28,7 +28,7 @@
# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
# POSSIBILITY OF SUCH DAMAGE.

import base, dependency_manager, logger, ik_ranking, planning, perception, simulation, tsr, viz
import base, controllers, dependency_manager, logger, ik_ranking, planning, perception, rave, simulation, tsr, viz
Copy link
Member

@mkoval mkoval Jul 6, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the controllers import - it will cause a regression of #308.

@mkoval
Copy link
Member

mkoval commented Jul 6, 2016

This looks good to me except for the one minor comment above.

@jeking04 Can you also confirm that this won't break anything?

@jeking04
Copy link
Contributor

jeking04 commented Jul 6, 2016

👍 looks good. this is useful additional functionality.

@athackst
Copy link
Author

athackst commented Jul 6, 2016

updated

@mkoval mkoval merged commit 068118a into personalrobotics:master Jul 7, 2016
@athackst athackst deleted the feature/tsr_gripper_library branch July 7, 2016 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants