{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2464","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2464/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2464/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2464/events","html_url":"https://github.com/RobotLocomotion/drake/pull/2464","id":157083895,"number":2464,"title":"add ellipsoids as another geometry type for the drake visualizer","user":{"login":"rdeits","id":591886,"avatar_url":"https://avatars.githubusercontent.com/u/591886?v=3","gravatar_id":"","url":"https://api.github.com/users/rdeits","html_url":"https://github.com/rdeits","followers_url":"https://api.github.com/users/rdeits/followers","following_url":"https://api.github.com/users/rdeits/following{/other_user}","gists_url":"https://api.github.com/users/rdeits/gists{/gist_id}","starred_url":"https://api.github.com/users/rdeits/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rdeits/subscriptions","organizations_url":"https://api.github.com/users/rdeits/orgs","repos_url":"https://api.github.com/users/rdeits/repos","events_url":"https://api.github.com/users/rdeits/events{/privacy}","received_events_url":"https://api.github.com/users/rdeits/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-05-26T21:31:33Z","updated_at":"2016-05-27T10:47:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RobotLocomotion/drake/pulls/2464","html_url":"https://github.com/RobotLocomotion/drake/pull/2464","diff_url":"https://github.com/RobotLocomotion/drake/pull/2464.diff","patch_url":"https://github.com/RobotLocomotion/drake/pull/2464.patch"},"body":"This PR is meant to accompany https://github.com/RobotLocomotion/director/pull/272 but either can be safely merged first. I've updated the drake visualizer to support ellipsoids as another geometry primitive. This will make it easier to draw ellipsoids, which will in turn make it easier to, for example, visualize inertial ellipsoids, without the LCMGL code currently used in https://github.com/RobotLocomotion/drake/blob/master/drake/systems/plants/BotVisualizer.m#L178\r\n\r\nNote that this modifies an LCM type definition, but because it's only adding an enum value and not modifying any fields, it should not change the LCM type fingerprint. It shouldn't affect compatibility with old LCM log files. \r\n\r\n@psiorx can you review this? :wink:\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"35\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/robotlocomotion/drake/2464)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/222117086","html_url":"https://github.com/RobotLocomotion/drake/pull/2464#issuecomment-222117086","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2464","id":222117086,"user":{"login":"RussTedrake","id":6442292,"avatar_url":"https://avatars.githubusercontent.com/u/6442292?v=3","gravatar_id":"","url":"https://api.github.com/users/RussTedrake","html_url":"https://github.com/RussTedrake","followers_url":"https://api.github.com/users/RussTedrake/followers","following_url":"https://api.github.com/users/RussTedrake/following{/other_user}","gists_url":"https://api.github.com/users/RussTedrake/gists{/gist_id}","starred_url":"https://api.github.com/users/RussTedrake/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RussTedrake/subscriptions","organizations_url":"https://api.github.com/users/RussTedrake/orgs","repos_url":"https://api.github.com/users/RussTedrake/repos","events_url":"https://api.github.com/users/RussTedrake/events{/privacy}","received_events_url":"https://api.github.com/users/RussTedrake/received_events","type":"User","site_admin":false},"created_at":"2016-05-27T10:47:24Z","updated_at":"2016-05-27T10:47:24Z","body":":LGTM:"}}
{"actor":{"display_login":"RussTedrake"}}