{"action":"created","issue":{"url":"https://api.github.com/repos/FEniCS/dolfinx/issues/143","repository_url":"https://api.github.com/repos/FEniCS/dolfinx","labels_url":"https://api.github.com/repos/FEniCS/dolfinx/issues/143/labels{/name}","comments_url":"https://api.github.com/repos/FEniCS/dolfinx/issues/143/comments","events_url":"https://api.github.com/repos/FEniCS/dolfinx/issues/143/events","html_url":"https://github.com/FEniCS/dolfinx/pull/143","id":345474544,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA0NjA4MzAy","number":143,"title":"Force Eigen 32-byte alignment for AVX compatibility","user":{"login":"w1th0utnam3","id":12967904,"node_id":"MDQ6VXNlcjEyOTY3OTA0","avatar_url":"https://avatars0.githubusercontent.com/u/12967904?v=4","gravatar_id":"","url":"https://api.github.com/users/w1th0utnam3","html_url":"https://github.com/w1th0utnam3","followers_url":"https://api.github.com/users/w1th0utnam3/followers","following_url":"https://api.github.com/users/w1th0utnam3/following{/other_user}","gists_url":"https://api.github.com/users/w1th0utnam3/gists{/gist_id}","starred_url":"https://api.github.com/users/w1th0utnam3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/w1th0utnam3/subscriptions","organizations_url":"https://api.github.com/users/w1th0utnam3/orgs","repos_url":"https://api.github.com/users/w1th0utnam3/repos","events_url":"https://api.github.com/users/w1th0utnam3/events{/privacy}","received_events_url":"https://api.github.com/users/w1th0utnam3/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-07-28T19:53:17Z","updated_at":"2018-07-29T08:08:09Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/FEniCS/dolfinx/pulls/143","html_url":"https://github.com/FEniCS/dolfinx/pull/143","diff_url":"https://github.com/FEniCS/dolfinx/pull/143.diff","patch_url":"https://github.com/FEniCS/dolfinx/pull/143.patch"},"body":"In order to use the underlying memory of Eigen matrices in functions that rely on AVX instructions, a Preprocessor directive has to be set (https://eigen.tuxfamily.org/dox/TopicPreprocessorDirectives.html).\r\n\r\nThe change in `cpp/CMakeLists.txt` ensures that the cpp library is compiled with the right Eigen configuration, the change in `cpp/cmake/templates/UseDOLFIN.cmake.in` ensures that any projects depending on Dolfin (including tests, demos and the python module) get compiled with the right settings."},"comment":{"url":"https://api.github.com/repos/FEniCS/dolfinx/issues/comments/408659744","html_url":"https://github.com/FEniCS/dolfinx/pull/143#issuecomment-408659744","issue_url":"https://api.github.com/repos/FEniCS/dolfinx/issues/143","id":408659744,"node_id":"MDEyOklzc3VlQ29tbWVudDQwODY1OTc0NA==","user":{"login":"jhale","id":2894451,"node_id":"MDQ6VXNlcjI4OTQ0NTE=","avatar_url":"https://avatars3.githubusercontent.com/u/2894451?v=4","gravatar_id":"","url":"https://api.github.com/users/jhale","html_url":"https://github.com/jhale","followers_url":"https://api.github.com/users/jhale/followers","following_url":"https://api.github.com/users/jhale/following{/other_user}","gists_url":"https://api.github.com/users/jhale/gists{/gist_id}","starred_url":"https://api.github.com/users/jhale/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhale/subscriptions","organizations_url":"https://api.github.com/users/jhale/orgs","repos_url":"https://api.github.com/users/jhale/repos","events_url":"https://api.github.com/users/jhale/events{/privacy}","received_events_url":"https://api.github.com/users/jhale/received_events","type":"User","site_admin":false},"created_at":"2018-07-29T08:08:09Z","updated_at":"2018-07-29T08:08:09Z","author_association":"MEMBER","body":"Hi Patrick,\n\nWe're discussing this in #simd on slack.\n\nJack\n\nOn Sat, 28 Jul 2018, 23:07 Fabian Löschner, <notifications@github.com>\nwrote:\n\n> @pefarrell <https://github.com/pefarrell> I don't have an AVX-512 CPU for\n> testing but I would assume it would suffice to just use the preprocessor\n> directive from this PR and replace 32 with 64?\n>\n> —\n> You are receiving this because you are subscribed to this thread.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/FEniCS/dolfinx/pull/143#issuecomment-408634968>, or mute\n> the thread\n> <https://github.com/notifications/unsubscribe-auth/ACwqc2uJmIWJBBzXX_C3c6lfURLIoQGFks5uLNKHgaJpZM4VlKcS>\n> .\n>\n"}}
{"actor":{"display_login":"jhale"}}