Bug report #9909

Prevent creation of recursive models (or provide solution to break out of loops)

Added by Anita Graser over 10 years ago. Updated almost 9 years ago.

Status:Closed
Priority:Normal
Assignee:Victor Olaya
Category:Processing/Modeller
Affected QGIS version:2.4.0 Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:
Crashes QGIS or corrupts data:No Copied to github as #:18423

Description

Currently, it is possible to include the model itself into the model workflow thus creating a model which calls itself. Of course this causes errors.

History

#1 Updated by Victor Olaya over 10 years ago

mmm, actually it shouldn't cause error. The modeler is designed to allow nested models. So it's a bug, but the solution is not removing models in the list of algorithms in the model, but fixing it.

#2 Updated by Anita Graser over 10 years ago

Sorry for being unclear :-)

It's great that modeler allows nested models! I just wanted to suggest that it might make sense to check if the user is creating circular references in his models. If model A calls model A to get to its results, it won't ever stop, or is there a way to break out of such loops?

#3 Updated by Giovanni Manghi over 10 years ago

  • Subject changed from It's possible to create recursive models to It's possible to stop recursive models?

#4 Updated by Anita Graser over 10 years ago

  • Subject changed from It's possible to stop recursive models? to Prevent creation of recursive models (or provide solution to break out of loops)

#5 Updated by Giovanni Manghi about 10 years ago

  • Category deleted (64)
  • Crashes QGIS or corrupts data set to No
  • Affected QGIS version set to 2.4.0
  • Project changed from 78 to QGIS Application

#6 Updated by Giovanni Manghi about 10 years ago

  • Category set to Processing/Modeller

#7 Updated by Victor Olaya almost 9 years ago

fixed in #a549ded47bf6bfebc050d64502ac3dc8b9e921d6

#8 Updated by Victor Olaya almost 9 years ago

  • Status changed from Open to Closed

Also available in: Atom PDF