{"action":"created","issue":{"url":"https://api.github.com/repos/mrdoob/three.js/issues/5059","labels_url":"https://api.github.com/repos/mrdoob/three.js/issues/5059/labels{/name}","comments_url":"https://api.github.com/repos/mrdoob/three.js/issues/5059/comments","events_url":"https://api.github.com/repos/mrdoob/three.js/issues/5059/events","html_url":"https://github.com/mrdoob/three.js/issues/5059","id":37737626,"number":5059,"title":"Vertex normals calculation in BufferGeometry","user":{"login":"ibachar","id":3380169,"avatar_url":"https://avatars.githubusercontent.com/u/3380169?v=3","gravatar_id":"","url":"https://api.github.com/users/ibachar","html_url":"https://github.com/ibachar","followers_url":"https://api.github.com/users/ibachar/followers","following_url":"https://api.github.com/users/ibachar/following{/other_user}","gists_url":"https://api.github.com/users/ibachar/gists{/gist_id}","starred_url":"https://api.github.com/users/ibachar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ibachar/subscriptions","organizations_url":"https://api.github.com/users/ibachar/orgs","repos_url":"https://api.github.com/users/ibachar/repos","events_url":"https://api.github.com/users/ibachar/events{/privacy}","received_events_url":"https://api.github.com/users/ibachar/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mrdoob/three.js/labels/Suggestion","name":"Suggestion","color":"ffffcf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2014-07-13T09:04:12Z","updated_at":"2015-07-04T05:59:45Z","closed_at":null,"body":"I think I've noticed two bugs with the calculation of vertex normals in BufferGeometry in r67.\r\n\r\n1. The face normal is calculated and added to the vertices that belong to that face but shouldn't the face normal be normalized first? Without normalization, there is an implicit weighting on the face normals.\r\n2. At the end of the function, shouldn't the line \"this.normalsNeedUpdate = true;\" be replaced with \"this.attributes.normal.needsUpdate = true;\"?"},"comment":{"url":"https://api.github.com/repos/mrdoob/three.js/issues/comments/118467290","html_url":"https://github.com/mrdoob/three.js/issues/5059#issuecomment-118467290","issue_url":"https://api.github.com/repos/mrdoob/three.js/issues/5059","id":118467290,"user":{"login":"WestLangley","id":1000017,"avatar_url":"https://avatars.githubusercontent.com/u/1000017?v=3","gravatar_id":"","url":"https://api.github.com/users/WestLangley","html_url":"https://github.com/WestLangley","followers_url":"https://api.github.com/users/WestLangley/followers","following_url":"https://api.github.com/users/WestLangley/following{/other_user}","gists_url":"https://api.github.com/users/WestLangley/gists{/gist_id}","starred_url":"https://api.github.com/users/WestLangley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WestLangley/subscriptions","organizations_url":"https://api.github.com/users/WestLangley/orgs","repos_url":"https://api.github.com/users/WestLangley/repos","events_url":"https://api.github.com/users/WestLangley/events{/privacy}","received_events_url":"https://api.github.com/users/WestLangley/received_events","type":"User","site_admin":false},"created_at":"2015-07-04T05:59:45Z","updated_at":"2015-07-04T05:59:45Z","body":"@mrdoob This fell through the cracks. What behavior would you prefer in (1) and (2)?"}}