Algomorph / InfiniTAM

A Framework for the Volumetric Integration of Depth Images (with some experimental code to handle dynamic scenes)
http://www.infinitam.org
Other
21 stars 3 forks source link

VTK version #136

Closed ziruiw-dev closed 5 years ago

ziruiw-dev commented 5 years ago

Hi @Algomorph,

I come here from the sobfu repo issue. I am trying to build your feature/dynamicfusion branch and got a cmake error about cannot find VTK. I installed the VTK7 with qt and python-vtk by

sudo apt-get install livbvtk7-qt-dev python-vtk6

and the cmake can find the vtk but gave a warning:

-- The imported target "vtk" references the file
   "/usr/bin/vtk"
but this file does not exist.  Possible reasons include:
* The file was deleted, renamed, or moved to another location.
* An install or uninstall procedure did not complete successfully.
* The installation package was faulty and contained
   "/usr/lib/cmake/vtk-7.1/VTKTargets.cmake"
but not all the files it references.

and when I build the project I got error:

InfiniTAM/ITMLib/Instantiations/VoxelBlockHash/../../Utils/Visualization/ITMSceneSliceVisualizer3D.tpp: In static member function ‘static void ITMLib::ITMSceneSliceVisualizer3D<TVoxelCanonical, TVoxelLive, TIndex>::SetUpSDFColorLookupTable(vtkSmartPointer<vtkLookupTable>&, const double*, const double*, const double*, const double*, const double*, const double*)’:
InfiniTAM/ITMLib/Instantiations/VoxelBlockHash/../../Utils/Visualization/ITMSceneSliceVisualizer3D.tpp:388:81: error: no matching function for call to ‘vtkLookupTable::SetTableValue(ITMLib::VoxelColorIndex, const double*&)’
  table->SetTableValue(POSITIVE_TRUNCATED_SDF_COLOR_INDEX, positiveTruncatedColor);

Which version of VTK are you using? with Qt I suppose? I assume you build from the source code? how did you configure the VTK if you build from source code?

thanks a lot.

Algomorph commented 5 years ago

Hi, @Ryan-Zirui-Wang ,

Algomorph commented 5 years ago

@Ryan-Zirui-Wang P.S. I've sent you an invite to collaborate directly on my fork, so you can commit directly here if you prefer and we can work together more easily.

ziruiw-dev commented 5 years ago

Thanks a lot for the invitation, I'll try to remove it and see what I can contribute.

I still would give my re-implementation some time so I'll try to compare my method and yours see if anything I did is not quite right... I hope it's alright for you?

Algomorph commented 5 years ago

@Ryan-Zirui-Wang sure, I'm open to anything, even if you want to just grab the features from mine and simply merge them into yours that's fine. I'd be more than happy to work with you on your repository.

Please take your time to compare both codebases and decide which course of action you find best.

A few points to keep in mind:

ziruiw-dev commented 5 years ago

Thanks a lot for the input! It's very well written and with nice documentation.

I have got it running after switching to Qt5 and VTK8 like you said without changing the code at all.

The dependencies are not too bad for now, VTK and Qt are a little bit fuzzy during the cmake configuration but it's very easy to use in general.