{"action":"created","issue":{"url":"https://api.github.com/repos/nipy/mindboggle/issues/67","labels_url":"https://api.github.com/repos/nipy/mindboggle/issues/67/labels{/name}","comments_url":"https://api.github.com/repos/nipy/mindboggle/issues/67/comments","events_url":"https://api.github.com/repos/nipy/mindboggle/issues/67/events","html_url":"https://github.com/nipy/mindboggle/pull/67","id":113739896,"number":67,"title":"Use updated API in VTK 6.3.0","user":{"login":"nicholsn","id":463344,"avatar_url":"https://avatars.githubusercontent.com/u/463344?v=3","gravatar_id":"","url":"https://api.github.com/users/nicholsn","html_url":"https://github.com/nicholsn","followers_url":"https://api.github.com/users/nicholsn/followers","following_url":"https://api.github.com/users/nicholsn/following{/other_user}","gists_url":"https://api.github.com/users/nicholsn/gists{/gist_id}","starred_url":"https://api.github.com/users/nicholsn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicholsn/subscriptions","organizations_url":"https://api.github.com/users/nicholsn/orgs","repos_url":"https://api.github.com/users/nicholsn/repos","events_url":"https://api.github.com/users/nicholsn/events{/privacy}","received_events_url":"https://api.github.com/users/nicholsn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-28T04:09:54Z","updated_at":"2015-10-28T16:02:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nipy/mindboggle/pulls/67","html_url":"https://github.com/nipy/mindboggle/pull/67","diff_url":"https://github.com/nipy/mindboggle/pull/67.diff","patch_url":"https://github.com/nipy/mindboggle/pull/67.patch"},"body":"This PR fixes what looks like a change to the VTK API. \r\n\r\nWhen running the VTK decimate functions using version 6.3.0 installed via conda on Mac and Ubuntu I received the following error:\r\n```\r\n...\r\nReduced 266798 to 6486 triangular faces\r\n  Scalar 35: 3368 vertices\r\n('Downsampling vtk files.',)\r\nLoad \"scalars\" scalars from lh_1.vtk\r\nTraceback (most recent call last):\r\n  File \"get_data.py\", line 46, in <module>\r\n    sample_rate=float(args['sample_rate']))\r\n  File \"/Users/nicholsn/Repos/roygbiv/roygbiv/__init__.py\", line 105, in freesurfer_annot_to_vtks\r\n    downsample_vtk(vtk_file, sample_rate=sample_rate)\r\n  File \"/Users/nicholsn/Repos/roygbiv/roygbiv/__init__.py\", line 28, in downsample_vtk\r\n    decimate_file(vtk_file, reduction=1 - sample_rate, output_vtk=vtk_file, save_vtk=True, smooth_steps=0)\r\n  File \"/Users/nicholsn/miniconda/envs/roygbiv/lib/python2.7/site-packages/mindboggle/guts/mesh.py\", line 940, in decimate_file\r\n    save_vtk, output_vtk)\r\n  File \"/Users/nicholsn/miniconda/envs/roygbiv/lib/python2.7/site-packages/mindboggle/guts/mesh.py\", line 830, in decimate\r\n    decimate.SetInput(polydata)\r\nAttributeError: SetInput\r\n```\r\nUpdating to use `SetInputData` seems to have fixed the error."},"comment":{"url":"https://api.github.com/repos/nipy/mindboggle/issues/comments/151893006","html_url":"https://github.com/nipy/mindboggle/pull/67#issuecomment-151893006","issue_url":"https://api.github.com/repos/nipy/mindboggle/issues/67","id":151893006,"user":{"login":"nicholsn","id":463344,"avatar_url":"https://avatars.githubusercontent.com/u/463344?v=3","gravatar_id":"","url":"https://api.github.com/users/nicholsn","html_url":"https://github.com/nicholsn","followers_url":"https://api.github.com/users/nicholsn/followers","following_url":"https://api.github.com/users/nicholsn/following{/other_user}","gists_url":"https://api.github.com/users/nicholsn/gists{/gist_id}","starred_url":"https://api.github.com/users/nicholsn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicholsn/subscriptions","organizations_url":"https://api.github.com/users/nicholsn/orgs","repos_url":"https://api.github.com/users/nicholsn/repos","events_url":"https://api.github.com/users/nicholsn/events{/privacy}","received_events_url":"https://api.github.com/users/nicholsn/received_events","type":"User","site_admin":false},"created_at":"2015-10-28T16:02:07Z","updated_at":"2015-10-28T16:02:07Z","body":"Ah, so this is also reflected in the cpp libs:\r\n\r\n```\r\n(roygbiv)07:47:50 (master) ~/Repos/mindboggle/surface_cpp_tools/bin$ make\r\nScanning dependencies of target FsSurfaceReader\r\n[  3%] Building CXX object CMakeFiles/FsSurfaceReader.dir/FsSurfaceReader.cpp.o\r\n/Users/nicholsn/Repos/mindboggle/surface_cpp_tools/FsSurfaceReader.cpp:109:9: error: no member named 'Update' in 'vtkPolyData'\r\n    pd->Update();\r\n    ~~  ^\r\n/Users/nicholsn/Repos/mindboggle/surface_cpp_tools/FsSurfaceReader.cpp:116:10: error: no member named 'SetInput' in 'vtkPolyDataNormals'; did you mean 'SetOutput'?\r\n    pdn->SetInput(pd);\r\n         ^~~~~~~~\r\n         SetOutput\r\n/Users/nicholsn/miniconda/envs/roygbiv/include/vtk-6.3/vtkPolyDataAlgorithm.h:49:16: note: 'SetOutput' declared here\r\n  virtual void SetOutput(vtkDataObject* d);\r\n```\r\nAfter correcting these, I'm now left with the following call to `Update` that is apparently no longer correct... Not sure what the fix is for this...\r\n```\r\n(roygbiv)08:04:49 {fix/decimate} ~/Repos/mindboggle/surface_cpp_tools/bin$ make\r\n[  3%] Building CXX object CMakeFiles/FsSurfaceReader.dir/FsSurfaceReader.cpp.o\r\n/Users/nicholsn/Repos/mindboggle/surface_cpp_tools/FsSurfaceReader.cpp:109:9: error: no member named 'Update' in 'vtkPolyData'\r\n    pd->Update();\r\n    ~~  ^\r\n1 error generated.\r\nmake[2]: *** [CMakeFiles/FsSurfaceReader.dir/FsSurfaceReader.cpp.o] Error 1\r\nmake[1]: *** [CMakeFiles/FsSurfaceReader.dir/all] Error 2\r\nmake: *** [all] Error 2\r\n```"}}