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 bush customizer #71

Merged
merged 4 commits into from
Aug 28, 2015
Merged

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Aug 26, 2015

bushのカスタマイザを追加しました。
sample1.wrlのブッシュありモデルをsample1_bush.wrlとして追加しました。

bushのパラメータは、
#58
のコメントも踏まえて、呼出側(BodyやModelLoaderUtil)から設定を与えるのは
仕様が大幅にかわりそうなのと複雑そうでしたので、
設定ファイルのパスを環境変数で与えて与えるようにしてみています。
(roslaunchからはenvタグでできます)
パラメータは、bushのための仮想関節名、spring, dumperが任意の数与えられます。

よろしくお願いします。

@snozawa
Copy link
Contributor Author

snozawa commented Aug 26, 2015

こちらのコードですが、おそらくVRMLに指定して設定ファイルを適切におかなければ
デフォルトでよみこまれても悪さはしないと思っています。

また、コード確認とパラメータの大部分は@YouheiKakiuchiさんにいただいてます。

@snozawa
Copy link
Contributor Author

snozawa commented Aug 27, 2015

設定ファイルがinstall先に含まれていなかったので、修正しました。
よろしくお願いします。

@snozawa
Copy link
Contributor Author

snozawa commented Aug 28, 2015

テストがとおってないので確認します。

@snozawa snozawa closed this Aug 28, 2015
@snozawa snozawa reopened this Aug 28, 2015
@snozawa
Copy link
Contributor Author

snozawa commented Aug 28, 2015

通りました。
お手すきのときにご確認いただけますと幸いです。

fkanehiro added a commit that referenced this pull request Aug 28, 2015
@fkanehiro fkanehiro merged commit d826323 into fkanehiro:master Aug 28, 2015
@snozawa snozawa deleted the add_bush_customizer branch September 5, 2015 18:21
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.

2 participants