{"action":"created","issue":{"url":"https://api.github.com/repos/cinder/Cinder/issues/924","labels_url":"https://api.github.com/repos/cinder/Cinder/issues/924/labels{/name}","comments_url":"https://api.github.com/repos/cinder/Cinder/issues/924/comments","events_url":"https://api.github.com/repos/cinder/Cinder/issues/924/events","html_url":"https://github.com/cinder/Cinder/pull/924","id":78252788,"number":924,"title":"Rewrite of the AxisAlignedBox3f class.","user":{"login":"paulhoux","id":304908,"avatar_url":"https://avatars.githubusercontent.com/u/304908?v=3","gravatar_id":"","url":"https://api.github.com/users/paulhoux","html_url":"https://github.com/paulhoux","followers_url":"https://api.github.com/users/paulhoux/followers","following_url":"https://api.github.com/users/paulhoux/following{/other_user}","gists_url":"https://api.github.com/users/paulhoux/gists{/gist_id}","starred_url":"https://api.github.com/users/paulhoux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paulhoux/subscriptions","organizations_url":"https://api.github.com/users/paulhoux/orgs","repos_url":"https://api.github.com/users/paulhoux/repos","events_url":"https://api.github.com/users/paulhoux/events{/privacy}","received_events_url":"https://api.github.com/users/paulhoux/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-19T21:14:15Z","updated_at":"2015-05-22T01:38:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cinder/Cinder/pulls/924","html_url":"https://github.com/cinder/Cinder/pull/924","diff_url":"https://github.com/cinder/Cinder/pull/924.diff","patch_url":"https://github.com/cinder/Cinder/pull/924.patch"},"body":"Complete rewrite of the AxisAlignedBox3f class. \r\n* its internal representation has been reduced to 2 ```vec3```'s.\r\n* most intersection tests now only use a few lines of code, thanks to the new internal structure and the use of ```glm::``` functions where possible.\r\n* I have tried to keep backward compatibility, tested all samples that use an AABB and wrote a few unit tests.\r\n* Added new functions, like ```contains(vec3)```, ```project(vec3, float *min, float *max)``` and ```intersect( AxisAlignedBox3f )```.\r\n* implementation is header-only, to allow the compiler to inline the code where possible.\r\n* Added ```transform()``` and ```transformed()``` methods to ```Area``` as well. The ```Rect``` class already had a ```transformCopy()``` method. After discussing this with Andrew, we have chosen to rename these to ```transformed()``` and change the name in ```Path2d``` and ```Shape2d``` as well."},"comment":{"url":"https://api.github.com/repos/cinder/Cinder/issues/comments/104471844","html_url":"https://github.com/cinder/Cinder/pull/924#issuecomment-104471844","issue_url":"https://api.github.com/repos/cinder/Cinder/issues/924","id":104471844,"user":{"login":"paulhoux","id":304908,"avatar_url":"https://avatars.githubusercontent.com/u/304908?v=3","gravatar_id":"","url":"https://api.github.com/users/paulhoux","html_url":"https://github.com/paulhoux","followers_url":"https://api.github.com/users/paulhoux/followers","following_url":"https://api.github.com/users/paulhoux/following{/other_user}","gists_url":"https://api.github.com/users/paulhoux/gists{/gist_id}","starred_url":"https://api.github.com/users/paulhoux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paulhoux/subscriptions","organizations_url":"https://api.github.com/users/paulhoux/orgs","repos_url":"https://api.github.com/users/paulhoux/repos","events_url":"https://api.github.com/users/paulhoux/events{/privacy}","received_events_url":"https://api.github.com/users/paulhoux/received_events","type":"User","site_admin":false},"created_at":"2015-05-22T01:38:22Z","updated_at":"2015-05-22T01:38:22Z","body":"Ok, I think it is ready to go. Let me know if you have any questions or concerns."}}