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

[jsk_naoqi_robot/README] Update for kinetic user #969

Merged
merged 6 commits into from
Nov 16, 2018

Conversation

kochigami
Copy link
Contributor

My PC was broken, so I had to set new environment for Pepper and NAO.
I followed this README, and I modified some parts which I thought information was not enough.

  • SDK path is different according to its version
  • for Pepper developer. more packages are required from source
  • delete duplicate information about rossetip

@kochigami kochigami added the naoqi label Oct 1, 2018
@kochigami kochigami closed this Oct 17, 2018
@kochigami kochigami reopened this Oct 17, 2018
@708yamaguchi
Copy link
Member

708yamaguchi commented Oct 17, 2018

@kochigami
I guess that you reopened this PR because travis test failed.
This test failed because the test log exceeded the maximum log length.

I am facing the same problem as this problem(#963 (comment)), and I am trying to solve this problem.

@kochigami
Copy link
Contributor Author

@708yamaguchi
Thank you for sharing the information!

@@ -37,12 +51,6 @@ rosdep install -y -r --from-paths src --ignore-src
catkin build
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Before executing catkin build, installing ros-$ROS_DISTRO-pepper(nao)-meshes manually is required in order to get pepper(nao).l

@kochigami
Copy link
Contributor Author

kochigami commented Oct 25, 2018

I also added explanations about installing mesh files to create nao.l and pepper.l.
I think gathering information is useful.

As a next step (not this pull-request), I have to check other readme such as naoeus for duplicate information about installing mesh files.

@kochigami
Copy link
Contributor Author

Travis passed.
Could you merge this when you have time?, or should I merge three commits into one by using rebase?


```
export ROS_IP="133.11.216.yyy"
catkin build
Copy link
Contributor Author

Choose a reason for hiding this comment

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

catkin build -c is better? because many packages are abandoned when one package failed to be compiled.
https://gist.github.com/kochigami/c29acd6ca562752d834996c6b51f89ad

@kochigami kochigami changed the title [jsk_naoqi_robot/README] add info about SDK path, more pkgs for step4 [jsk_naoqi_robot/README] Update for kinetic user Nov 2, 2018
@kochigami
Copy link
Contributor Author

Travis passed.
I hope this readme has enough information to set environment for naoqi robots.

@k-okada
Could you merge this when you have time?

@k-okada k-okada merged commit b849e41 into jsk-ros-pkg:master Nov 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants