mollynnobrien / b-tk

Automatically exported from code.google.com/p/b-tk
0 stars 0 forks source link

Various use of vtkTransform2D - "switch to vtk 5.8 / 5.10" #32

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Use latest release (BTK-0.1.10_src)
2. Configure (might need patch from issue #31)
3. make

What is the expected output? What do you see instead?
[...]
Scanning dependencies of target btkAppendEvent
[  7%] Building CXX object 
Wrapping/Octave/CMakeFiles/btkAppendEvent.dir/__/Matlab/btkAppendEvent.cpp.o
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp: In member function 
‘virtual bool btk::VTKAxis::Paint(vtkContext2D*)’:
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:280:17: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:281:15: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:305:21: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:306:19: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:322:21: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:323:19: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:339:21: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:340:19: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:356:21: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:357:19: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp: In member function 
‘double btk::VTKAxis::CalculateNiceMinMax(double&, double&)’:
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:518:15: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
<code_to_btk>/BTK/Code/VisSupport/VTK/btkVTKAxis.cpp:519:13: error: ‘class 
btk::VTKAxis’ has no member named ‘GetTransform’
make[2]: *** [Code/VisSupport/VTK/CMakeFiles/BTKVTK.dir/btkVTKAxis.cpp.o] Error 
1
make[1]: *** [Code/VisSupport/VTK/CMakeFiles/BTKVTK.dir/all] Error 2

What version of the product are you using? On what operating system?
BTK 0.1.10 source code, Ubuntu 12.04

Please provide any additional information below.
Attached is an attempt to add missing member variables in multiple places. It 
allows to build Mokka and it works mostly, however Charts seem to be broken.

Original issue reported on code.google.com by martin.f...@gmail.com on 21 Jun 2012 at 2:59

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by arnaud.barre on 21 Jun 2012 at 3:23

GoogleCodeExporter commented 8 years ago
In case my mail is never received...

This error is related to a major modification in the Charts API in VTK 5.8 and 
greater. For example, the GetTransform method was removed. Currently only the 
branch of BTK named vtk5100 can be compiled with VTK 5.10, VTK 5.8 could be 
also used, but was not tested. Even if all the code can be compiled (Mokka 
included), some parts of the charts are still broken.

As a first solution with the sources of BTK-0.1.10, it is better to downgrade 
the version of VTK used and go back to VTK 5.6.1. This problem will be resolved 
in BTK-0.2.

Original comment by arnaud.barre on 21 Jun 2012 at 3:26

GoogleCodeExporter commented 8 years ago
Okay. Feel free to close this issue as it is only relevant for the current 
development stage and the transition to 5.10. Maybe adding an issue entitled 
"switch to vtk 5.10" would be more clear.

Just a side note: the stable version of Ubuntu (12.04), which will be supported 
until 2017 comes with vtk 5.8 in its package system. If Mokka could be compiled 
against this, this would enable people to install btk without having to 
manually download and install vtk 5.10.

Original comment by martin.f...@gmail.com on 26 Jun 2012 at 8:32

GoogleCodeExporter commented 8 years ago
I didn't try to compile the branch vtk5100 with VTK 5.8, but it should work as 
far as I know. I think I removed all the classes proposed only in VTK 5.10.

Original comment by arnaud.barre on 27 Jun 2012 at 9:41

GoogleCodeExporter commented 8 years ago
Since the revision r1208, the branch vtk-5100 can be compiled with VTK 5.8. 
After several tests, everything works as expected. Only a small glitch was 
found when an item is dropped into a chart.

Note: The code was tested under Windows XP 32-bit, but should work under Ubuntu 
12.04.

Original comment by arnaud.barre on 29 Aug 2012 at 6:45

GoogleCodeExporter commented 8 years ago
This issue was fixed with the reintegration of the branch vtk5100 in the trunk 
(see revision r1257).

Original comment by arnaud.barre on 23 Sep 2012 at 6:14