{"action":"closed","issue":{"url":"https://api.github.com/repos/phetsims/trig-tour/issues/33","labels_url":"https://api.github.com/repos/phetsims/trig-tour/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/phetsims/trig-tour/issues/33/comments","events_url":"https://api.github.com/repos/phetsims/trig-tour/issues/33/events","html_url":"https://github.com/phetsims/trig-tour/issues/33","id":102549159,"number":33,"title":"Some confusing documentation in TrigTourModel","user":{"login":"jessegreenberg","id":6396244,"avatar_url":"https://avatars.githubusercontent.com/u/6396244?v=3","gravatar_id":"","url":"https://api.github.com/users/jessegreenberg","html_url":"https://github.com/jessegreenberg","followers_url":"https://api.github.com/users/jessegreenberg/followers","following_url":"https://api.github.com/users/jessegreenberg/following{/other_user}","gists_url":"https://api.github.com/users/jessegreenberg/gists{/gist_id}","starred_url":"https://api.github.com/users/jessegreenberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jessegreenberg/subscriptions","organizations_url":"https://api.github.com/users/jessegreenberg/orgs","repos_url":"https://api.github.com/users/jessegreenberg/repos","events_url":"https://api.github.com/users/jessegreenberg/events{/privacy}","received_events_url":"https://api.github.com/users/jessegreenberg/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"dubson","id":5090870,"avatar_url":"https://avatars.githubusercontent.com/u/5090870?v=3","gravatar_id":"","url":"https://api.github.com/users/dubson","html_url":"https://github.com/dubson","followers_url":"https://api.github.com/users/dubson/followers","following_url":"https://api.github.com/users/dubson/following{/other_user}","gists_url":"https://api.github.com/users/dubson/gists{/gist_id}","starred_url":"https://api.github.com/users/dubson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dubson/subscriptions","organizations_url":"https://api.github.com/users/dubson/orgs","repos_url":"https://api.github.com/users/dubson/repos","events_url":"https://api.github.com/users/dubson/events{/privacy}","received_events_url":"https://api.github.com/users/dubson/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-08-22T16:31:16Z","updated_at":"2015-09-25T00:23:59Z","closed_at":"2015-09-25T00:23:59Z","body":"Some of the documentation in TrigTourModel is a little confusing or contradictory.\r\n\r\nIn ```setFullAngleInRadians()```, documentation says \r\n> set the full angle, the small angle and various turns counts, given the current full angle\r\n\r\nIt looks like this function is setting the current full angle from a desired full angle, which is different from setting the full angle from the current full angle.  \r\n\r\nIn ```setAngle()```, documentation says\r\n> set the full angle, and various turns counts, given the current small angle\r\n\r\nBut the first thing this function does is set the small angle.  It looks like this function sets all angles given a desired small angle, so the documentation is misleading.   It might also be helpful to rename setAngle() to something like setModelAngles() or setAnglesFromSmallAngle().\r\n\r\nWould you mind clarifying the documentation for these functions?"}}