From b4616b7381404cc8760ed81acf3667f6c188b5d5 Mon Sep 17 00:00:00 2001 From: Patrick Avery Date: Thu, 9 Apr 2020 17:22:33 -0400 Subject: [PATCH] Do not emit data modified after pipeline finishes There is a new datasource every time now, so this is not necessary. Additionally, this is causing crashing when closing the application when operators are present, and it is also causing crashing occasionally when deleting datasources that have operators. Signed-off-by: Patrick Avery --- tomviz/PipelineModel.cxx | 8 -------- 1 file changed, 8 deletions(-) diff --git a/tomviz/PipelineModel.cxx b/tomviz/PipelineModel.cxx index e84c84136..a98e36e24 100644 --- a/tomviz/PipelineModel.cxx +++ b/tomviz/PipelineModel.cxx @@ -777,14 +777,6 @@ void PipelineModel::dataSourceAdded(DataSource* dataSource) connect(pipeline, &Pipeline::operatorAdded, this, &PipelineModel::operatorAdded, Qt::UniqueConnection); - // Fire signal to indicate that the transformed data source has been modified - // when the pipeline has been executed. - // TODO This should probably be move else where! - connect(pipeline, &Pipeline::finished, [this, pipeline]() { - auto transformed = pipeline->transformedDataSource(); - emit dataSourceModified(transformed); - }); - // When restoring a data source from a state file it will have its operators // before we can listen to the signal above. Display those operators. foreach (auto op, dataSource->operators()) {