Bad module dependency for vtkGenerateTimeSteps
Description
The useful vtkGenerateTimeSteps
filter can be found in the VTK::FiltersHybrid
module. Several sources could need this filter to avoid code duplication, if they wanted the source to allow time steps generation. This is the case of the vtkTimeSourceExample
and the more recent vtkSpatioTemporalHarmonicsSource
.
However, the VTK::FiltersHybrid
module implies adding strong dependencies, such as the VTK::RenderingCore
module. This would be bad to need the rendering module to be able to create sources.
Possible fixes
From the best to the worst solution imo:
- The
vtkGenerateTimeSteps
filter should be moved out of this module, in a better fitting module such asVTK::FiltersCore
orVTK::FiltersGeneral
. - The
VTK::FiltersHybrid
module should move every filter depending on rendering in a more appropriate module, to remove theVTK::RenderingCore
module dependency. - Sources needing the
vtkGenerateTimeSteps
filter should be moved in theVTK::FiltersHybrid
module.