Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #244 +/- ##
==========================================
+ Coverage 94.52% 94.70% +0.18%
==========================================
Files 15 15
Lines 3379 3383 +4
==========================================
+ Hits 3194 3204 +10
+ Misses 185 179 -6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
pnuu
left a comment
There was a problem hiding this comment.
LGTM.
So the Manager didn't work when it was a global varible, and the cache usage fixes it?
Yes, that's correct, in python >= 3.14, the process spawning method changed, so the memory context isn't shared at all between processes anymore. I older version we could provoke the error too if we change the default spawning method to "forkserver" (which is now the default). |
With new handling of multiprocessing and forking, python 3.14 broke a few things in trollflow2.
This PR fixes it.