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

compile wrl file to collada model #9

Merged
merged 1 commit into from
Aug 28, 2014

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Aug 26, 2014

https://github.com/start-jsk/openhrp3/blob/master/patch/CompileColladaModel.patch

wrlのモデルをdaeに変換しています.
他の変換すべきモデルファイルがあれば教えて下さい.

@k-okada
Copy link
Contributor Author

k-okada commented Aug 27, 2014

こちら,マージされないのはなにか問題がありましたでしょうか? @fkanehiro

@fkanehiro
Copy link
Owner

いえ、「教えてください」とあるので、誰か何か答えるのかなぁ、と。

@k-okada
Copy link
Contributor Author

k-okada commented Aug 27, 2014

なるほど、了解です。もうちょっと待ちましょう。
みんな、見てる〜? (^O^)/

@k-okada k-okada mentioned this pull request Aug 27, 2014
@snozawa
Copy link
Contributor

snozawa commented Aug 28, 2014

(申し訳ありません、遅くなりました)

ちなみにこのモデルの変換はどういった目的で行われていますでしょうか。
例えば、modelloaderのテストのためでしょうか。
export-colladaがOpenHRP3でサポートされてる機能なので、
dae変換バージョンも生成しておく、というのであれば、
きかい的にsample/model以下の全モデルを変換するのはいかがでしょうか。
(wrlの中には、main.wrlなものからinlineされてるのがあるので、それは除外する必要がありそうですが、、)

そうでない場合は、downstreamなところでsample1.wrlのdaeを作っているので、
それもこちらの時点で作成いただけると良さそうに思いますので、追加をお願いできればと思います。

@k-okada k-okada force-pushed the compile_wrl_to_collada branch from b2008d4 to 8de74e5 Compare August 28, 2014 09:33
@k-okada k-okada force-pushed the compile_wrl_to_collada branch from 8de74e5 to 82f4102 Compare August 28, 2014 09:36
@k-okada
Copy link
Contributor Author

k-okada commented Aug 28, 2014

そもそもは,openhrp3にmodelloaderのテストを加えたらdaeのファイルも必要になったという経緯です.

そうでない場合は、downstreamなところでsample1.wrlのdaeを作っているので、
それもこちらの時点で作成いただけると良さそうに思いますので、追加をお願いできればと思います。

がよく分からなかったけど,これでよいでしょうか? @snozawa

#10 が必要です.

@snozawa
Copy link
Contributor

snozawa commented Aug 28, 2014

はい、変更後のPRのようなものをイメージしていました。

fkanehiro added a commit that referenced this pull request Aug 28, 2014
compile wrl file to collada model
@fkanehiro fkanehiro merged commit 5d8109e into fkanehiro:master Aug 28, 2014
@k-okada k-okada deleted the compile_wrl_to_collada branch October 19, 2014 03:12
k-okada pushed a commit to k-okada/openhrp3 that referenced this pull request Feb 6, 2015
k-okada pushed a commit to k-okada/openhrp3 that referenced this pull request Feb 6, 2015
k-okada pushed a commit to k-okada/openhrp3 that referenced this pull request Feb 6, 2015
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