DYN-9965: Change in PythonNet3 syntax

DSPythonNet3/DSPythonNet3Evaluator.cs

@@ -20,7 +20,7 @@
     internal class DynamoCPythonHandleComparer : IEqualityComparer<DynamoCPythonHandle>
     {
 
         public bool Equals(DynamoCPythonHandle x, DynamoCPythonHandle y)
         {
             return x.PythonObjectID.Equals(y.PythonObjectID);
         }
@@ -74,7 +74,7 @@
             try
             {
                 var pyobj = DSPythonNet3Evaluator.globalScope.Get(PythonObjectID.ToString());
                 return pyobj.ToString();
             }
             catch (Exception e)
             {
@@ -143,8 +143,8 @@
             { // Session is null when running unit tests.
                 if (ExecutionEvents.ActiveSession != null)
                 {
                     dynamoLogger = ExecutionEvents.ActiveSession.GetParameterValue(ParameterKeys.Logger) as DynamoLogger;
                     return dynamoLogger;
                 }
                 return dynamoLogger;
             }
@@ -799,6 +799,61 @@
         public override event EvaluationFinishedEventHandler EvaluationFinished;
 
         private bool registeredUnwrapMarshaler;
+        private const string ConnectionNodeCompatSentinel = "__dspynet3_connnode_compat__";
+
+        /// <summary>
+        /// Compatibility shim for "node-first" static methods exposed by some Dynamo/Revit libraries.
+        /// In older engines these could be invoked like instance methods (e.g. node.SubNodesOfSize(2)).
+        /// Python.NET 3 binds static methods strictly, so we patch a small set of known APIs to keep
+        /// existing graphs working.
+        /// </summary>
+        private static string ConnectionNodeCompatPatchCode()
+        {
+            return $@"
+import builtins as __builtins__
+try:
+    from AdvanceSteel.ConnectionAutomation.Nodes import ConnectionNode as __ConnectionNode
+except Exception:
+    __ConnectionNode = None
+if __ConnectionNode is not None:
+    if not getattr(__builtins__, '{ConnectionNodeCompatSentinel}', False):
+        def __dspynet3__patch_connnode_instance(__obj):
+            # Attach instance-callable wrappers for node-first static methods.
+            try:
+                if not isinstance(__obj, __ConnectionNode):
+                    return
+                def __subnodes(__n, *args, __obj=__obj, **kwargs):
+                    return __ConnectionNode.SubNodesOfSize(__obj, __n, *args, **kwargs)
+                def __existing(*args, __obj=__obj, **kwargs):
+                    return __ConnectionNode.ExistingConnections(__obj, *args, **kwargs)
+                try:
+                    setattr(__obj, 'SubNodesOfSize', __subnodes)
+                except Exception:
+                    pass
+                try:
+                    setattr(__obj, 'ExistingConnections', __existing)
+                except Exception:
+                    pass
+            except Exception:
+                pass
+        def __dspynet3__walk_and_patch(__x):
+            try:
+                if isinstance(__x, (list, tuple)):
+                    for __i in __x:
+                        __dspynet3__walk_and_patch(__i)
+                else:
+                    __dspynet3__patch_connnode_instance(__x)
+            except Exception:
+                pass
+        setattr(__builtins__, '__dspynet3__walk_and_patch_connnode', __dspynet3__walk_and_patch)
+        setattr(__builtins__, '{ConnectionNodeCompatSentinel}', True)
+    try:
+        __inp = globals().get('IN', None)
+        __builtins__.__dspynet3__walk_and_patch_connnode(__inp)
+    except Exception:
+        pass
+";
+        }
 
         /// <summary>
         /// Called immediately before evaluation starts
@@ -844,6 +899,8 @@
 
                 registeredUnwrapMarshaler = true;
             }
+
+            scope.Exec(ConnectionNodeCompatPatchCode());
         }
 
         /// <summary>