{"action":"created","issue":{"url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/issues/183","repository_url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK","labels_url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/issues/183/labels{/name}","comments_url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/issues/183/comments","events_url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/issues/183/events","html_url":"https://github.com/InsightSoftwareConsortium/ITK/pull/183","id":380889060,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMwOTgzNDg2","number":183,"title":"ENH: Add ThirdParty module Eigen3","user":{"login":"phcerdan","id":3021667,"node_id":"MDQ6VXNlcjMwMjE2Njc=","avatar_url":"https://avatars1.githubusercontent.com/u/3021667?v=4","gravatar_id":"","url":"https://api.github.com/users/phcerdan","html_url":"https://github.com/phcerdan","followers_url":"https://api.github.com/users/phcerdan/followers","following_url":"https://api.github.com/users/phcerdan/following{/other_user}","gists_url":"https://api.github.com/users/phcerdan/gists{/gist_id}","starred_url":"https://api.github.com/users/phcerdan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phcerdan/subscriptions","organizations_url":"https://api.github.com/users/phcerdan/orgs","repos_url":"https://api.github.com/users/phcerdan/repos","events_url":"https://api.github.com/users/phcerdan/events{/privacy}","received_events_url":"https://api.github.com/users/phcerdan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-11-14T21:13:17Z","updated_at":"2018-11-18T01:58:24Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/pulls/183","html_url":"https://github.com/InsightSoftwareConsortium/ITK/pull/183","diff_url":"https://github.com/InsightSoftwareConsortium/ITK/pull/183.diff","patch_url":"https://github.com/InsightSoftwareConsortium/ITK/pull/183.patch"},"body":"The idea is to introduce Eigen3 into ITK.\r\nPlease note that ITKEigen is already a module.\r\nThe name of this module is ITKEigen3\r\n\r\nIt uses a modified Eigen3 stored in:\r\nhttps://gitlab.kitware.com/third-party/eigen\r\n\r\nI forked and modified the existing branch `for/vtk`\r\ninto `for/itk`.\r\n\r\n-- Note that for development I am using my own fork:\r\nhttps://gitlab.kitware.com/phcerdan/eigen/tree/for/itk --\r\n\r\nA MR is in process requesting to create the branch for/itk\r\nhttps://gitlab.kitware.com/third-party/eigen/merge_requests/1\r\n\r\n'for/itk' differs from 'for/vtk' that it allows the developer\r\nto use find_package(Eigen3) in the internal itkeigen, just\r\npopulating the Eigen3_DIR with the internal folder:\r\n\r\n```cmake\r\nset(_internal_cmake_eigen3)\r\nlist(GET ITKEigen3_INCLUDE_DIRS 0 _internal_cmake_eigen3)\r\nset(Eigen3_DIR \"${_internal_cmake_eigen3}/itkeigen\")\r\n```\r\nSee https://github.com/phcerdan/ITKTotalVariation for up-to-date\r\nhandling of Eigen3_DIR.\r\n\r\nEigen3 is faster than non-optimized implementations of\r\nlapack. VNL has the netlib (non-optimized) CLAPACK implementation.\r\nCLAPACK is the c-version of the netlib lapack libraries (written in fortran),\r\nit used the f2c conversor from fortran to c. It is not maintained anymore.\r\n\r\nOne of the purposes on introduced Eigen3 is to speed up eigen\r\ndecomposition. Also having the most used linear algebra library\r\nin c++ opens opportunities for bridging with other libraries.\r\n\r\nThe first application for the module is to use it in a external remote\r\nmodule TotalVariation, that performs total variation regularization for\r\ndenoising methods:\r\nhttps://github.com/phcerdan/ITKTotalVariation\r\nThe module is a wrap around the third party library:\r\nUpstream: https://github.com/albarji/proxTV\r\nUsed fork: https://github.com/phcerdan/proxTV/tree/use_eigen\r\nthat requires Eigen3."},"comment":{"url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/issues/comments/439661586","html_url":"https://github.com/InsightSoftwareConsortium/ITK/pull/183#issuecomment-439661586","issue_url":"https://api.github.com/repos/InsightSoftwareConsortium/ITK/issues/183","id":439661586,"node_id":"MDEyOklzc3VlQ29tbWVudDQzOTY2MTU4Ng==","user":{"login":"kwrobot-v1[bot]","id":44507454,"node_id":"MDM6Qm90NDQ1MDc0NTQ=","avatar_url":"https://avatars0.githubusercontent.com/in/19744?v=4","gravatar_id":"","url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D","html_url":"https://github.com/apps/kwrobot-v1","followers_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/followers","following_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/repos","events_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/kwrobot-v1%5Bbot%5D/received_events","type":"Bot","site_admin":false},"created_at":"2018-11-18T01:58:24Z","updated_at":"2018-11-18T01:58:24Z","author_association":"NONE","body":"Errors:\n\n  - Failed to run the checks: `service error`."}}
{"actor":{"display_login":"kwrobot-v1"}}