{"action":"created","issue":{"url":"https://api.github.com/repos/libigl/libigl/issues/71","labels_url":"https://api.github.com/repos/libigl/libigl/issues/71/labels{/name}","comments_url":"https://api.github.com/repos/libigl/libigl/issues/71/comments","events_url":"https://api.github.com/repos/libigl/libigl/issues/71/events","html_url":"https://github.com/libigl/libigl/pull/71","id":71967322,"number":71,"title":"adding writeOBJPoly aligned with readOBJPoly","user":{"login":"chrisidefix","id":382555,"avatar_url":"https://avatars.githubusercontent.com/u/382555?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisidefix","html_url":"https://github.com/chrisidefix","followers_url":"https://api.github.com/users/chrisidefix/followers","following_url":"https://api.github.com/users/chrisidefix/following{/other_user}","gists_url":"https://api.github.com/users/chrisidefix/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisidefix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisidefix/subscriptions","organizations_url":"https://api.github.com/users/chrisidefix/orgs","repos_url":"https://api.github.com/users/chrisidefix/repos","events_url":"https://api.github.com/users/chrisidefix/events{/privacy}","received_events_url":"https://api.github.com/users/chrisidefix/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-04-29T19:33:44Z","updated_at":"2015-04-30T14:31:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/libigl/libigl/pulls/71","html_url":"https://github.com/libigl/libigl/pull/71","diff_url":"https://github.com/libigl/libigl/pull/71.diff","patch_url":"https://github.com/libigl/libigl/pull/71.patch"},"body":"Added a writeOBJPoly function to support writing out mixed element meshes (triangles, quads, polygons, ..), which is comparable to readOBJPoly for reading such meshes."},"comment":{"url":"https://api.github.com/repos/libigl/libigl/issues/comments/97818958","html_url":"https://github.com/libigl/libigl/pull/71#issuecomment-97818958","issue_url":"https://api.github.com/repos/libigl/libigl/issues/71","id":97818958,"user":{"login":"alecjacobson","id":2241689,"avatar_url":"https://avatars.githubusercontent.com/u/2241689?v=3","gravatar_id":"","url":"https://api.github.com/users/alecjacobson","html_url":"https://github.com/alecjacobson","followers_url":"https://api.github.com/users/alecjacobson/followers","following_url":"https://api.github.com/users/alecjacobson/following{/other_user}","gists_url":"https://api.github.com/users/alecjacobson/gists{/gist_id}","starred_url":"https://api.github.com/users/alecjacobson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alecjacobson/subscriptions","organizations_url":"https://api.github.com/users/alecjacobson/orgs","repos_url":"https://api.github.com/users/alecjacobson/repos","events_url":"https://api.github.com/users/alecjacobson/events{/privacy}","received_events_url":"https://api.github.com/users/alecjacobson/received_events","type":"User","site_admin":false},"created_at":"2015-04-30T14:31:01Z","updated_at":"2015-04-30T14:31:01Z","body":"For now, let's use the matrix_to_list version. I did a quick benchmark and\nit doesn't make _that_ much of a difference. I guess ideally we could write\na functional version that provides a \"matrix\" (either Eigen:Matrix* or\nvector<vector> ) as an input argument and an `at(i,j)` function handle as a\n_template_ argument. But that's a little complicated for now.\n\nPlease do test on many example meshes beforehand.\n\nThanks!\n\nOn Thu, Apr 30, 2015 at 9:46 AM, chrisidefix <notifications@github.com>\nwrote:\n\n> Sure, I guess the question is - what do you prefer - redundant code that\n> you have to edit twice in case you would like to make a change or faster\n> execution time...\n>\n> I did test the code, but my test case contained 4 faces ... [image:\n> :1234:] and therefore worked, despite of this obvious mistake.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/libigl/libigl/pull/71#issuecomment-97798107>.\n>\n"}}