Merge branch 'main' into mh14779/numba-csp-v1

Author: timkpaine

.github/workflows/conda.yml

@@ -78,7 +78,7 @@ jobs:
         - name: Checkout
           uses: actions/checkout@v6
 
-        - uses: mamba-org/setup-micromamba@v2
+        - uses: mamba-org/setup-micromamba@v3
           with:
             micromamba-version: '2.3.2-0'
             environment-file: conda/dev-environment-unix.yml
@@ -89,7 +89,7 @@ jobs:
             post-cleanup: 'all'
           if: ${{ runner.os != 'Windows' }}
 
-        - uses: mamba-org/setup-micromamba@v2
+        - uses: mamba-org/setup-micromamba@v3
           with:
             micromamba-version: '2.3.2-0'
             environment-file: conda/dev-environment-win.yml

cpp/csp/python/PyNode.cpp

@@ -307,8 +307,12 @@ void PyNode::executeImpl()
 
             if( tsinputTicked( idx ) || passiveConvert )
             {
-                Py_XDECREF( *m_localVars[ idx ] );
+                //Note this is intentionally kept in a temp and decref-ed after updating m_localVars[idx] since
+                //lastValueToPython can trigger a GC Collect which can reach into generator vars ( localVars ) which would be nulled out
+                PyObject * old = *m_localVars[ idx ];
                 *m_localVars[ idx ] = lastValueToPython( ts );
+                Py_XDECREF( old );
+                
                 if( passiveConvert )
                     m_passiveCounts[ idx ] = count;
             }