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 scripts for auto generating controller config yaml #440

Merged

Conversation

YoheiKakiuchi
Copy link
Member

robot_controller_config.yamlをrobot.yamlのlimbから自動生成するようにしました。
コミットされているrobot_controller_config.yamlがある場合は、上書きはしません。
start-jsk/rtmros_gazebo#38

@k-okada
Copy link
Member

k-okada commented Apr 22, 2014

@YoheiKakiuchi can you push last commit ? (or you can merge last 2 commits and push again), because jekins jobs take over the "buidl check" -> http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/rtmros-common-check/DEBIAN_REPOSITORY=ros-shadow-fixed,ROS_DISTRO=hydro,slave=ec2/121/

@YoheiKakiuchi
Copy link
Member Author

travis でlogファイル容量エラーで止まっています。 どの場合どういう対処法がありますか?

@k-okada
Copy link
Member

k-okada commented Apr 26, 2014

エラーがあると、エラーを出すためのコードが走り大量のログをだします

エラーがなければ容量内に収まると思います。

2014年4月26日土曜日、Yohei [email protected]さんは書きました:

travis でlogファイル容量エラーで止まっています。 どの場合どういう対処法がありますか?


Reply to this email directly or view it on GitHubhttps://github.com//pull/440#issuecomment-41462579
.

@snozawa
Copy link
Contributor

snozawa commented Apr 26, 2014

基本方針としては、travisで走るようなテストプログラムも、
普通にしてたら極力出力しないようにするかんじでしょうか。
euslispのunittest.lの出力も減ってますか?

@k-okada
Copy link
Member

k-okada commented Apr 26, 2014

unittest.lは
euslisp/EusLisp@577acb3
でちょっと表示をへらしました。

安直に/dev/nullに飛ばすのもあるけど結局エラーの時は見たくなるので、無限ループに思います。

最初のrosdep installで結構メッセージがでているのでよりquietなモードがあるといいんだけど。

YoheiKakiuchi added a commit that referenced this pull request Apr 30, 2014
コンパイルが通っているようなのでマージします。
@YoheiKakiuchi YoheiKakiuchi merged commit c29da69 into start-jsk:master Apr 30, 2014
@k-okada
Copy link
Member

k-okada commented Apr 30, 2014

compile_robot_model.cmake is obviously redundant and need to re-write and clean up for further development and code maintenance. so before you add new code in there, please consider to volunteer to working on that.

Also pleease review the new proposal on code stability -> #449, before add new code in the repository.

@YoheiKakiuchi YoheiKakiuchi deleted the add_controller_config_yaml branch May 14, 2014 14:10
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

Successfully merging this pull request may close these issues.

3 participants