{"action":"created","issue":{"url":"https://api.github.com/repos/simbody/simbody/issues/354","labels_url":"https://api.github.com/repos/simbody/simbody/issues/354/labels{/name}","comments_url":"https://api.github.com/repos/simbody/simbody/issues/354/comments","events_url":"https://api.github.com/repos/simbody/simbody/issues/354/events","html_url":"https://github.com/simbody/simbody/pull/354","id":64662920,"number":354,"title":"Decorative geom opensim40","user":{"login":"aymanhab","id":3972208,"avatar_url":"https://avatars.githubusercontent.com/u/3972208?v=3","gravatar_id":"","url":"https://api.github.com/users/aymanhab","html_url":"https://github.com/aymanhab","followers_url":"https://api.github.com/users/aymanhab/followers","following_url":"https://api.github.com/users/aymanhab/following{/other_user}","gists_url":"https://api.github.com/users/aymanhab/gists{/gist_id}","starred_url":"https://api.github.com/users/aymanhab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aymanhab/subscriptions","organizations_url":"https://api.github.com/users/aymanhab/orgs","repos_url":"https://api.github.com/users/aymanhab/repos","events_url":"https://api.github.com/users/aymanhab/events{/privacy}","received_events_url":"https://api.github.com/users/aymanhab/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-03-27T00:20:24Z","updated_at":"2015-04-09T23:42:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/simbody/simbody/pulls/354","html_url":"https://github.com/simbody/simbody/pull/354","diff_url":"https://github.com/simbody/simbody/pull/354.diff","patch_url":"https://github.com/simbody/simbody/pull/354.patch"},"body":"This commit adds DecorativeTorus and DecorativeArrow to the set of primitives that can be handled by the Visualizer. As of now the built in visualizer ignores these primitives. The OpenSim GUI/Visualizer which uses the same mechanism/primitives will eventually use these to display forces, torques and WrapObjects."},"comment":{"url":"https://api.github.com/repos/simbody/simbody/issues/comments/91382261","html_url":"https://github.com/simbody/simbody/pull/354#issuecomment-91382261","issue_url":"https://api.github.com/repos/simbody/simbody/issues/354","id":91382261,"user":{"login":"sherm1","id":4088016,"avatar_url":"https://avatars.githubusercontent.com/u/4088016?v=3","gravatar_id":"","url":"https://api.github.com/users/sherm1","html_url":"https://github.com/sherm1","followers_url":"https://api.github.com/users/sherm1/followers","following_url":"https://api.github.com/users/sherm1/following{/other_user}","gists_url":"https://api.github.com/users/sherm1/gists{/gist_id}","starred_url":"https://api.github.com/users/sherm1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sherm1/subscriptions","organizations_url":"https://api.github.com/users/sherm1/orgs","repos_url":"https://api.github.com/users/sherm1/repos","events_url":"https://api.github.com/users/sherm1/events{/privacy}","received_events_url":"https://api.github.com/users/sherm1/received_events","type":"User","site_admin":false},"created_at":"2015-04-09T23:42:22Z","updated_at":"2015-04-09T23:42:22Z","body":"> This's because arrows are normally used to display changing quantities ...\r\n\r\nThat is also true of `DecorativeLine` (at least as currently used). If you are using an arrow to represent a vector quantity v, it is easier and cheaper to provide startPoint and endPoint=startPoint+v than it is to calculate length=norm(v), direction=v/length especially since that requires checking for zero length or very small vectors whose direction is indeterminate. In contrast, startPoint+v is always well defined and works fine for v=0. And since this is now inconsistent with DecorativeLine, I would like to see DecorativeArrow changed to match.\r\n\r\nThanks for the clarification on `Hide`; sounds fine. Should be easy enough to implement in the API visualizer!\r\n"}}