{"action":"created","issue":{"url":"https://api.github.com/repos/s-nakaoka/choreonoid/issues/18","labels_url":"https://api.github.com/repos/s-nakaoka/choreonoid/issues/18/labels{/name}","comments_url":"https://api.github.com/repos/s-nakaoka/choreonoid/issues/18/comments","events_url":"https://api.github.com/repos/s-nakaoka/choreonoid/issues/18/events","html_url":"https://github.com/s-nakaoka/choreonoid/pull/18","id":67507869,"number":18,"title":"モデル入出力部分の強化","user":{"login":"yosuke","id":18067,"avatar_url":"https://avatars.githubusercontent.com/u/18067?v=3","gravatar_id":"","url":"https://api.github.com/users/yosuke","html_url":"https://github.com/yosuke","followers_url":"https://api.github.com/users/yosuke/followers","following_url":"https://api.github.com/users/yosuke/following{/other_user}","gists_url":"https://api.github.com/users/yosuke/gists{/gist_id}","starred_url":"https://api.github.com/users/yosuke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yosuke/subscriptions","organizations_url":"https://api.github.com/users/yosuke/orgs","repos_url":"https://api.github.com/users/yosuke/repos","events_url":"https://api.github.com/users/yosuke/events{/privacy}","received_events_url":"https://api.github.com/users/yosuke/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-04-10T04:41:03Z","updated_at":"2015-04-20T05:34:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/s-nakaoka/choreonoid/pulls/18","html_url":"https://github.com/s-nakaoka/choreonoid/pull/18","diff_url":"https://github.com/s-nakaoka/choreonoid/pull/18.diff","patch_url":"https://github.com/s-nakaoka/choreonoid/pull/18.patch"},"body":"OpenHRP形式のVRMLを出力するための強化と、ロードしたモデルからVRMLを復元するための機能付加を行いました。\r\nOpenHRP形式の出力に関しては入力部と同じくProtoInstanseを使うようにしようかなとも思ったのですが、フィールド名のtypoが怖かったのでヘッダに定義する形にしてあります。"},"comment":{"url":"https://api.github.com/repos/s-nakaoka/choreonoid/issues/comments/94360238","html_url":"https://github.com/s-nakaoka/choreonoid/pull/18#issuecomment-94360238","issue_url":"https://api.github.com/repos/s-nakaoka/choreonoid/issues/18","id":94360238,"user":{"login":"s-nakaoka","id":7353639,"avatar_url":"https://avatars.githubusercontent.com/u/7353639?v=3","gravatar_id":"","url":"https://api.github.com/users/s-nakaoka","html_url":"https://github.com/s-nakaoka","followers_url":"https://api.github.com/users/s-nakaoka/followers","following_url":"https://api.github.com/users/s-nakaoka/following{/other_user}","gists_url":"https://api.github.com/users/s-nakaoka/gists{/gist_id}","starred_url":"https://api.github.com/users/s-nakaoka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/s-nakaoka/subscriptions","organizations_url":"https://api.github.com/users/s-nakaoka/orgs","repos_url":"https://api.github.com/users/s-nakaoka/repos","events_url":"https://api.github.com/users/s-nakaoka/events{/privacy}","received_events_url":"https://api.github.com/users/s-nakaoka/received_events","type":"User","site_admin":false},"created_at":"2015-04-20T05:34:07Z","updated_at":"2015-04-20T05:34:07Z","body":"ありがとうございます、これはモデルエディタ用の改良ですね。\r\n\r\n> OpenHRP形式の出力に関しては入力部と同じくProtoInstanseを使うようにしようかなとも思ったのですが、フィールド名のtypoが怖かったのでヘッダに定義する形にしてあります。\r\n\r\nすみません、これについて少し気になりました。\r\n\r\nsrc/Util内のクラスは基本的にロボット関連の機能とは切り離して使えるよう設計していて、VRML関連もいちおうVRMLの仕様の範囲内のものとしています。したがって、OpenHRPのモデルファイルで拡張したノードはできればここには含めたくないところです。ProtoInstanceで実現できるのであればその方が望ましいですし、クラスとしての定義が必要だとしてもOpenHRPのノードはsrc/Body内で追加定義するといった対応でできれば有り難いです。「フィールド名のtypoが怖かったのでヘッダに定義する形にして」というのが具体的にどういうことか、OpenHRPの拡張ノードをsrc/Utilに書く必要があるか、といったことについて、気になりました。\r\n\r\n以上について検討・考慮しながら、必要であれば src/Util側のAPIを拡張してノードクラスの定義も追加できるようにする？といった対応ができればと思いますので、ご検討よろしくお願いします。"}}