Remove redundant scoped = False arguments to Signal.connect[Front]()

apps/execute/execute-1.py

@@ -171,7 +171,7 @@ def _run( self, args ) :
 
 		with context :
 			for node in nodes :
-				node.errorSignal().connect( Gaffer.WeakMethod( self.__error ), scoped = False )
+				node.errorSignal().connect( Gaffer.WeakMethod( self.__error ) )
 				try :
 					node["task"].executeSequence( frames )
 				except Exception as exception :

doc/source/WorkingWithThePythonScriptingAPI/TutorialStartupConfig1/index.md

@@ -64,7 +64,7 @@ Copy this code to a new a `customVariables.py` file in `~/gaffer/startup/gui`:
 
     	Gaffer.MetadataAlgo.setReadOnly( variables["projectResources"]["name"], True )
 
-    application.root()["scripts"].childAddedSignal().connect( __scriptAdded, scoped = False )
+    application.root()["scripts"].childAddedSignal().connect( __scriptAdded )
 ```
 
 Let's break down what's going on here.
@@ -125,7 +125,7 @@ We wrap up the config by adding our function to an event signal that fires when
 .. code-block:: python
     :lineno-start: 17
 
-    application.root()["scripts"].childAddedSignal().connect( __scriptAdded, scoped = False )
+    application.root()["scripts"].childAddedSignal().connect( __scriptAdded )
 ```
 
 Notice the use of the `application` variable. This is a special variable that refers to the startup config's parent application.

python/GafferCortexUI/ClassVectorParameterValueWidget.py

@@ -240,7 +240,7 @@ def _headerWidget( self ) :
 		with GafferUI.ListContainer( GafferUI.ListContainer.Orientation.Horizontal ) as result :
 
 			collapseButton = GafferUI.Button( image = "collapsibleArrowRight.png", hasFrame=False )
-			collapseButton.clickedSignal().connect( Gaffer.WeakMethod( self.__collapseButtonClicked ), scoped = False )
+			collapseButton.clickedSignal().connect( Gaffer.WeakMethod( self.__collapseButtonClicked ) )
 
 			GafferUI.Spacer( imath.V2i( 2 ) )
 
@@ -293,8 +293,8 @@ def _headerWidget( self ) :
 						self.__parameterHandler.childParameterHandler( self.__parameterHandler.parameter()["label"] ),
 					),
 				)
-				self.__label.buttonPressSignal().connect( Gaffer.WeakMethod( self.__labelButtonPress ), scoped = False )
-				self.getPlug().node().plugSetSignal().connect( Gaffer.WeakMethod( self.__plugSet ), scoped = False )
+				self.__label.buttonPressSignal().connect( Gaffer.WeakMethod( self.__labelButtonPress ) )
+				self.getPlug().node().plugSetSignal().connect( Gaffer.WeakMethod( self.__plugSet ) )
 
 			# parameters after the label
 			for parameter in headerParameters :

python/GafferCortexUI/CompoundVectorParameterValueWidget.py

@@ -97,8 +97,8 @@ def _headerWidget( self ) :
 			sizeEditable = sizeEditable,
 		)
 
-		self.__vectorDataWidget.editSignal().connect( Gaffer.WeakMethod( self.__edit ), scoped = False )
-		self.__vectorDataWidget.dataChangedSignal().connect( Gaffer.WeakMethod( self.__dataChanged ), scoped = False )
+		self.__vectorDataWidget.editSignal().connect( Gaffer.WeakMethod( self.__edit ) )
+		self.__vectorDataWidget.dataChangedSignal().connect( Gaffer.WeakMethod( self.__dataChanged ) )
 
 		self._requestUpdateFromValues()
 
@@ -302,4 +302,4 @@ def __parameterPopupMenu( menuDefinition, parameterValueWidget ) :
 			},
 		)
 
-GafferCortexUI.ParameterValueWidget.popupMenuSignal().connect( __parameterPopupMenu, scoped = False )
+GafferCortexUI.ParameterValueWidget.popupMenuSignal().connect( __parameterPopupMenu )

python/GafferCortexUI/OpPathPreview.py

@@ -55,7 +55,7 @@ def __init__( self, path ) :
 			# we'll replace this with the op in _deferredUpdate()
 			GafferUI.Spacer( imath.V2i( 1 ) )
 			button = GafferUI.Button( "Launch" )
-			button.clickedSignal().connect( Gaffer.WeakMethod( self.__executeClicked ), scoped = False )
+			button.clickedSignal().connect( Gaffer.WeakMethod( self.__executeClicked ) )
 
 		self._updateFromPath()
 

python/GafferCortexUI/ParameterPresets.py

@@ -502,4 +502,4 @@ def __parameterPopupMenu( menuDefinition, parameterValueWidget ) :
 	menuDefinition.append( "/Load Preset...", { "command" : IECore.curry( __loadPreset, parameterHandler ), "active" : editable } )
 	menuDefinition.append( "/Delete Presets...", { "command" : IECore.curry( __deletePresets, parameterHandler ) } )
 
-GafferCortexUI.ParameterValueWidget.popupMenuSignal().connect( __parameterPopupMenu, scoped = False )
+GafferCortexUI.ParameterValueWidget.popupMenuSignal().connect( __parameterPopupMenu )

python/GafferCortexUI/ParameterValueWidget.py

@@ -142,7 +142,7 @@ def __plugPopupMenu( menuDefinition, plugValueWidget ) :
 
 	ParameterValueWidget.popupMenuSignal()( menuDefinition, parameterValueWidget )
 
-GafferUI.PlugValueWidget.popupMenuSignal().connect( __plugPopupMenu, scoped = False )
+GafferUI.PlugValueWidget.popupMenuSignal().connect( __plugPopupMenu )
 
 # add menu items for presets
 

python/GafferCortexUI/ParameterisedHolderUI.py

@@ -103,7 +103,7 @@ def __init__( self, node ) :
 
 		self.__node = node
 		self.__window = None
-		self.clickedSignal().connect( Gaffer.WeakMethod( self.__clicked ), scoped = False )
+		self.clickedSignal().connect( Gaffer.WeakMethod( self.__clicked ) )
 
 	def getToolTip( self ) :
 

python/GafferCyclesUI/CyclesOptionsUI.py

@@ -1489,7 +1489,7 @@ def __init__( self, plugs, **kw ) :
 
 		GafferUI.PresetsPlugValueWidget.__init__( self, plugs, **kw )
 
-		self.getPlug().node().plugSetSignal().connect( Gaffer.WeakMethod( self.__plugSet ), scoped = False )
+		self.getPlug().node().plugSetSignal().connect( Gaffer.WeakMethod( self.__plugSet ) )
 
 	def __plugSet( self, plug ) :
 

python/GafferDispatch/LocalDispatcher.py

@@ -108,7 +108,7 @@ def __init__( self, batch, dispatcher ) :
 
 			self.__messageHandler = _MessageHandler()
 			self.__messagesChangedSignal = Gaffer.Signal1()
-			self.__messageHandler.messagesChangedSignal().connect( Gaffer.WeakMethod( self.__messagesChanged, fallbackResult = None ), scoped = False )
+			self.__messageHandler.messagesChangedSignal().connect( Gaffer.WeakMethod( self.__messagesChanged, fallbackResult = None ) )
 
 			self.__initBatchWalk( batch )
 

python/GafferDispatchTest/DispatcherTest.py

@@ -2274,7 +2274,7 @@ def testPostDispatchSignalSuccess( self ) :
 		self.assertEqual( postDispatchSlot[0], ( script["dispatcher"], True ) )
 
 		preDispatchConnection = GafferDispatch.Dispatcher.preDispatchSignal().connect(
-			lambda dispatcher : True, scoped = False
+			lambda dispatcher : True
 		)
 
 		script["dispatcher"]["task"].execute()
@@ -2300,17 +2300,17 @@ def testDispatchSignalShutdownCrash( self ) :
 
 		subprocess.check_call( [
 			Gaffer.executablePath(), "env", "python", "-c",
-			"""import GafferDispatch; GafferDispatch.Dispatcher.preDispatchSignal().connect( lambda d : True, scoped = False )"""
+			"""import GafferDispatch; GafferDispatch.Dispatcher.preDispatchSignal().connect( lambda d : True )"""
 		] )
 
 		subprocess.check_call( [
 			Gaffer.executablePath(), "env", "python", "-c",
-			"""import GafferDispatch; GafferDispatch.Dispatcher.dispatchSignal().connect( lambda d : None, scoped = False )"""
+			"""import GafferDispatch; GafferDispatch.Dispatcher.dispatchSignal().connect( lambda d : None )"""
 		] )
 
 		subprocess.check_call( [
 			Gaffer.executablePath(), "env", "python", "-c",
-			"""import GafferDispatch; GafferDispatch.Dispatcher.postDispatchSignal().connect( lambda d, s : None, scoped = False )"""
+			"""import GafferDispatch; GafferDispatch.Dispatcher.postDispatchSignal().connect( lambda d, s : None )"""
 		] )
 
 	def testAccessTaskNodeInSetupPlugs( self ) :

python/GafferDispatchTest/LocalDispatcherTest.py

@@ -1069,9 +1069,9 @@ def statusChanged( job ) :
 
 		def jobAdded( job ) :
 
-			job.statusChangedSignal().connect( statusChanged, scoped = False )
+			job.statusChangedSignal().connect( statusChanged )
 
-		script["dispatcher"].jobPool().jobAddedSignal().connect( jobAdded, scoped = False )
+		script["dispatcher"].jobPool().jobAddedSignal().connect( jobAdded )
 
 		# Test foreground dispatch
 

python/GafferDispatchUI/DispatchDialogue.py

@@ -104,7 +104,7 @@ def __init__( self, tasks, dispatchers, nodesToShow, postDispatchBehaviour=PostD
 						self.__dispatchersMenu = GafferUI.MultiSelectionMenu( allowMultipleSelection = False, allowEmptySelection = False )
 						self.__dispatchersMenu.append( [ x.getName() for x in self.__dispatchers ] )
 						self.__dispatchersMenu.setSelection( [ self.__dispatchers[0].getName() ] )
-						self.__dispatchersMenu.selectionChangedSignal().connect( Gaffer.WeakMethod( self.__dispatcherChanged ), scoped = False )
+						self.__dispatchersMenu.selectionChangedSignal().connect( Gaffer.WeakMethod( self.__dispatcherChanged ) )
 						dispatcherMenuColumn.setVisible( len(self.__dispatchers) > 1 )
 
 					self.__dispatcherFrame = GafferUI.Frame( borderStyle=GafferUI.Frame.BorderStyle.None_, borderWidth=0 )
@@ -125,10 +125,10 @@ def __init__( self, tasks, dispatchers, nodesToShow, postDispatchBehaviour=PostD
 				self.__messageWidget = GafferUI.MessageWidget( toolbars = True )
 				# connect to the collapsible state change so we can increase the window
 				# size when the details pane is first shown.
-				self.__messageCollapsibleConnection = self.__messageCollapsible.stateChangedSignal().connect( Gaffer.WeakMethod( self.__messageCollapsibleChanged ), scoped = False )
+				self.__messageCollapsibleConnection = self.__messageCollapsible.stateChangedSignal().connect( Gaffer.WeakMethod( self.__messageCollapsibleChanged ) )
 
 		self.__backButton = self._addButton( "Back" )
-		self.__backButton.clickedSignal().connectFront( Gaffer.WeakMethod( self.__initiateSettings ), scoped = False )
+		self.__backButton.clickedSignal().connectFront( Gaffer.WeakMethod( self.__initiateSettings ) )
 
 		self.__primaryButton = self._addButton( "Dispatch" )
 

python/GafferDispatchUI/DispatcherUI.py

@@ -203,7 +203,7 @@ def __init__( self, node, **kw ) :
 		GafferUI.Button.__init__( self, "Dispatch", **kw )
 
 		self.__node = node
-		self.clickedSignal().connect( Gaffer.WeakMethod( self.__clicked ), scoped = False )
+		self.clickedSignal().connect( Gaffer.WeakMethod( self.__clicked ) )
 
 	def __clicked( self, button ) :
 
@@ -283,7 +283,7 @@ def __init__( self, node, **kw ) :
 					# Dummy `gaffer://...` URL is sufficient to trigger `linkActivatedSignal`, which is all we need.
 					f'<a href="gaffer://createDispatcher"><font color={textColor}>Create a dispatcher</font></a> <font color={textColor}>to execute this node</font>',
 				)
-				label.linkActivatedSignal().connect( Gaffer.WeakMethod( self.__linkActivated ), scoped = False )
+				label.linkActivatedSignal().connect( Gaffer.WeakMethod( self.__linkActivated ) )
 
 				GafferUI.Spacer( size = imath.V2i( 0 ), parenting = { "expand" : True } )
 

python/GafferDispatchUI/LocalJobs.py

@@ -222,14 +222,14 @@ def __init__( self, scriptNode, **kw ) :
 					selectionMode = GafferUI.PathListingWidget.SelectionMode.Rows,
 				)
 				self.__jobListingWidget._qtWidget().header().setSortIndicator( 1, QtCore.Qt.AscendingOrder )
-				self.__jobListingWidget.selectionChangedSignal().connect( Gaffer.WeakMethod( self.__jobSelectionChanged ), scoped = False )
+				self.__jobListingWidget.selectionChangedSignal().connect( Gaffer.WeakMethod( self.__jobSelectionChanged ) )
 
 				with GafferUI.ListContainer( GafferUI.ListContainer.Orientation.Horizontal, spacing=5 ) :
 					GafferUI.Spacer( imath.V2i( 0 ), parenting = { "expand" : True } )
 					self.__killButton = GafferUI.Button( "Kill Selected Jobs" )
-					self.__killButton.clickedSignal().connect( Gaffer.WeakMethod( self.__killClicked ), scoped = False )
+					self.__killButton.clickedSignal().connect( Gaffer.WeakMethod( self.__killClicked ) )
 					self.__removeButton = GafferUI.Button( "Remove Selected Jobs" )
-					self.__removeButton.clickedSignal().connect( Gaffer.WeakMethod( self.__removeClicked ), scoped = False )
+					self.__removeButton.clickedSignal().connect( Gaffer.WeakMethod( self.__removeClicked ) )
 
 			with GafferUI.TabbedContainer() :
 
@@ -255,14 +255,14 @@ def __init__( self, scriptNode, **kw ) :
 
 		# Connecting to the JobPool and Job signals allows us to update our PathListingWidget
 		# immediately when jobs are added and removed or their status changes.
-		jobPool.jobAddedSignal().connect( Gaffer.WeakMethod( self.__jobAdded ), scoped = False )
-		jobPool.jobRemovedSignal().connect( Gaffer.WeakMethod( self.__jobRemoved ), scoped = False )
+		jobPool.jobAddedSignal().connect( Gaffer.WeakMethod( self.__jobAdded ) )
+		jobPool.jobRemovedSignal().connect( Gaffer.WeakMethod( self.__jobRemoved ) )
 
 		# But we also want to perform periodic updates to sample CPU/memory statistics, which
 		# we do using this timer.
 		self.__statisticsTimer = QtCore.QTimer()
 		self.__statisticsTimer.timeout.connect( Gaffer.WeakMethod( self.__statisticsTimeout ) )
-		self.visibilityChangedSignal().connect( Gaffer.WeakMethod( self.__visibilityChanged ), scoped = False )
+		self.visibilityChangedSignal().connect( Gaffer.WeakMethod( self.__visibilityChanged ) )
 
 		self.__updateButtons()
 
@@ -280,7 +280,7 @@ def __visibilityChanged( self, widget ) :
 	def __jobAdded( self, job ) :
 
 		assert( threading.current_thread() is threading.main_thread() )
-		job.statusChangedSignal().connect( Gaffer.WeakMethod( self.__jobStatusChanged ), scoped = False )
+		job.statusChangedSignal().connect( Gaffer.WeakMethod( self.__jobStatusChanged ) )
 		self.__jobListingWidget.getPath()._emitPathChanged()
 
 	def __jobRemoved( self, job ) :

python/GafferDispatchUI/PythonCommandUI.py

@@ -141,13 +141,13 @@ def __init__( self, plug, **kw ) :
 		self.__codeWidget.setHighlighter( GafferUI.CodeWidget.PythonHighlighter() )
 		self.__codeWidget.setCommentPrefix( "#" )
 
-		self.__codeWidget.editingFinishedSignal().connect( Gaffer.WeakMethod( self.__setPlugValue ), scoped = False )
+		self.__codeWidget.editingFinishedSignal().connect( Gaffer.WeakMethod( self.__setPlugValue ) )
 
 		self._addPopupMenu( self.__codeWidget )
 
 		node = self.__pythonCommandNode()
 		if node is not None :
-			node.plugDirtiedSignal().connect( Gaffer.WeakMethod( self.__pythonCommandPlugDirtied ), scoped = False )
+			node.plugDirtiedSignal().connect( Gaffer.WeakMethod( self.__pythonCommandPlugDirtied ) )
 		self.__updateCompleter()
 
 	def _updateFromValues( self, values, exception ) :

python/GafferDispatchUI/WedgeUI.py

@@ -309,7 +309,7 @@ def __init__( self, previewWidget, node, **kw ) :
 		previewWidget.setToolTip( "The values generated by the wedge" )
 
 		self.__node = node
-		node.plugDirtiedSignal().connect( Gaffer.WeakMethod( self.__plugDirtied ), scoped = False )
+		node.plugDirtiedSignal().connect( Gaffer.WeakMethod( self.__plugDirtied ) )
 
 	def node( self ) :
 

python/GafferImageTest/DisplayTest.py

@@ -323,12 +323,12 @@ def testSignalShutdownCrash( self ) :
 
 		subprocess.check_call( [
 			Gaffer.executablePath(), "env", "python", "-c",
-			"""import GafferImage; GafferImage.Display.driverCreatedSignal().connect( lambda d, p : None, scoped = False )"""
+			"""import GafferImage; GafferImage.Display.driverCreatedSignal().connect( lambda d, p : None )"""
 		] )
 
 		subprocess.check_call( [
 			Gaffer.executablePath(), "env", "python", "-c",
-			"""import GafferImage; GafferImage.Display.imageReceivedSignal().connect( lambda p : None, scoped = False )"""
+			"""import GafferImage; GafferImage.Display.imageReceivedSignal().connect( lambda p : None )"""
 		] )
 
 	def __testTransferImage( self, fileName ) :

python/GafferImageUI/CatalogueUI.py

@@ -316,8 +316,8 @@ def __init__( self, title ) :
 
 		Column.__init__( self, title )
 
-		self.buttonPressSignal().connect( Gaffer.WeakMethod( self.__buttonPress ), scoped = False )
-		self.buttonReleaseSignal().connect( Gaffer.WeakMethod( self.__buttonRelease ), scoped = False )
+		self.buttonPressSignal().connect( Gaffer.WeakMethod( self.__buttonPress ) )
+		self.buttonReleaseSignal().connect( Gaffer.WeakMethod( self.__buttonRelease ) )
 
 	def headerData( self, canceller = None ) :
 
@@ -492,7 +492,7 @@ def addCatalogueHotkeys( editor ) :
 	if not isinstance( editor, GafferUI.Viewer ) :
 		return
 
-	editor.keyPressSignal().connect( __viewerKeyPress, scoped = False )
+	editor.keyPressSignal().connect( __viewerKeyPress )
 
 def __viewerKeyPress( viewer, event ) :
 
@@ -739,47 +739,47 @@ def __init__( self, plug, **kw ) :
 				self.__pathListing.setDragPointer( "" )
 				self.__pathListing.setHeaderVisible( True )
 				self.__pathListing.selectionChangedSignal().connect(
-					Gaffer.WeakMethod( self.__pathListingSelectionChanged ), scoped = False
+					Gaffer.WeakMethod( self.__pathListingSelectionChanged )
 				)
 				self.__pathListing.dragEnterSignal().connect(
-					Gaffer.WeakMethod( self.__pathListingDragEnter ), scoped = False
+					Gaffer.WeakMethod( self.__pathListingDragEnter )
 				)
 				self.__pathListing.dragLeaveSignal().connect(
-					Gaffer.WeakMethod( self.__pathListingDragLeave ), scoped = False
+					Gaffer.WeakMethod( self.__pathListingDragLeave )
 				)
 				self.__pathListing.dragMoveSignal().connect(
-					Gaffer.WeakMethod( self.__pathListingDragMove ), scoped = False
+					Gaffer.WeakMethod( self.__pathListingDragMove )
 				)
 				self.__pathListing.dropSignal().connect(
-					Gaffer.WeakMethod( self.__pathListingDrop ), scoped = False
+					Gaffer.WeakMethod( self.__pathListingDrop )
 				)
 				self.__pathListing.getPath().pathChangedSignal().connect(
-					Gaffer.WeakMethod( self.__pathChanged ), scoped = False
+					Gaffer.WeakMethod( self.__pathChanged )
 				)
-				self.keyPressSignal().connect( Gaffer.WeakMethod( self.__keyPress ), scoped = False )
+				self.keyPressSignal().connect( Gaffer.WeakMethod( self.__keyPress ) )
 
 				with GafferUI.ListContainer( GafferUI.ListContainer.Orientation.Horizontal, spacing = 4 ) as self.__buttonRow :
 
 					addButton = GafferUI.Button( image = "pathChooser.png", hasFrame = False, toolTip = "Load image" )
-					addButton.clickedSignal().connect( Gaffer.WeakMethod( self.__addClicked ), scoped = False )
+					addButton.clickedSignal().connect( Gaffer.WeakMethod( self.__addClicked ) )
 
 					self.__duplicateButton = GafferUI.Button( image = "duplicate.png", hasFrame = False, toolTip = "Duplicate selected image, hold <kbd>alt</kbd> to view copy. [<kbd>Ctrl-D</kbd>]" )
 					self.__duplicateButton.setEnabled( False )
-					self.__duplicateButton.clickedSignal().connect( Gaffer.WeakMethod( self.__duplicateClicked ), scoped = False )
+					self.__duplicateButton.clickedSignal().connect( Gaffer.WeakMethod( self.__duplicateClicked ) )
 
 					self.__exportButton = GafferUI.Button( image = "export.png", hasFrame = False, toolTip = "Export selected image" )
 					self.__exportButton.setEnabled( False )
-					self.__exportButton.clickedSignal().connect( Gaffer.WeakMethod( self.__exportClicked ), scoped = False )
+					self.__exportButton.clickedSignal().connect( Gaffer.WeakMethod( self.__exportClicked ) )
 
 					self.__extractButton = GafferUI.Button( image = "extract.png", hasFrame = False, toolTip = "Create CatalogueSelect node for selected image" )
 					self.__extractButton.setEnabled( False )
-					self.__extractButton.clickedSignal().connect( Gaffer.WeakMethod( self.__extractClicked ), scoped = False )
+					self.__extractButton.clickedSignal().connect( Gaffer.WeakMethod( self.__extractClicked ) )
 
 					GafferUI.Spacer( imath.V2i( 0 ), parenting = { "expand" : True } )
 
 					self.__removeButton = GafferUI.Button( image = "delete.png", hasFrame = False, toolTip = "Remove selected image [<kbd>Delete</kbd>]" )
 					self.__removeButton.setEnabled( False )
-					self.__removeButton.clickedSignal().connect( Gaffer.WeakMethod( self.__removeClicked ), scoped = False )
+					self.__removeButton.clickedSignal().connect( Gaffer.WeakMethod( self.__removeClicked ) )
 
 				GafferUI.Spacer( size = imath.V2i( 2 ) )
 
@@ -800,9 +800,9 @@ def __init__( self, plug, **kw ) :
 
 		self.__mergeGroupId = 0
 
-		Gaffer.Metadata.plugValueChangedSignal( plug.node() ).connect( Gaffer.WeakMethod( self.__plugMetadataValueChanged ), scoped = False )
+		Gaffer.Metadata.plugValueChangedSignal( plug.node() ).connect( Gaffer.WeakMethod( self.__plugMetadataValueChanged ) )
 
-		self.contextMenuSignal().connect( Gaffer.WeakMethod( self.__contextMenu ), scoped = False )
+		self.contextMenuSignal().connect( Gaffer.WeakMethod( self.__contextMenu ) )
 
 	def buttonRow( self ):
 

python/GafferImageUI/CreateViewsUI.py

@@ -130,7 +130,7 @@ def __init__( self, plug ) :
 			GafferUI.Spacer( imath.V2i( GafferUI.PlugWidget.labelWidth(), 1 ) )
 
 			self.__button = GafferUI.Button( image = "plus.png", hasFrame = False )
-			self.__button.clickedSignal().connect( Gaffer.WeakMethod( self.__buttonClicked ), scoped = False )
+			self.__button.clickedSignal().connect( Gaffer.WeakMethod( self.__buttonClicked ) )
 
 			GafferUI.Spacer( imath.V2i( 1 ), imath.V2i( 999999, 1 ), parenting = { "expand" : True } )