Skip to content

Commit b4616b7

Browse files
committed
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 <patrick.avery@kitware.com>
1 parent 4482381 commit b4616b7

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

tomviz/PipelineModel.cxx

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -777,14 +777,6 @@ void PipelineModel::dataSourceAdded(DataSource* dataSource)
777777
connect(pipeline, &Pipeline::operatorAdded, this,
778778
&PipelineModel::operatorAdded, Qt::UniqueConnection);
779779

780-
// Fire signal to indicate that the transformed data source has been modified
781-
// when the pipeline has been executed.
782-
// TODO This should probably be move else where!
783-
connect(pipeline, &Pipeline::finished, [this, pipeline]() {
784-
auto transformed = pipeline->transformedDataSource();
785-
emit dataSourceModified(transformed);
786-
});
787-
788780
// When restoring a data source from a state file it will have its operators
789781
// before we can listen to the signal above. Display those operators.
790782
foreach (auto op, dataSource->operators()) {

0 commit comments

Comments
 (0)