-
Notifications
You must be signed in to change notification settings - Fork 167
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JP-2980 Documentation improvement for multiprocessing: a script which spawns processes on import will cause system failure #8408
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8408 +/- ##
==========================================
+ Coverage 75.31% 75.76% +0.45%
==========================================
Files 474 476 +2
Lines 38965 39404 +439
==========================================
+ Hits 29345 29854 +509
+ Misses 9620 9550 -70
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Co-authored-by: Howard Bushouse <[email protected]>
Co-authored-by: Howard Bushouse <[email protected]>
Co-authored-by: Howard Bushouse <[email protected]>
Co-authored-by: Brett Graham <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checked the docs built from the CI test in this PR and they look good.
@penaguerrero Does this also close #8404 or is it just related? |
it is not related. In a conversation with Brett we discussed that a cause for the memory issue could be the multiple re-openings of datamodels. He is looking into that separately. |
Resolves JP-2980
Closes #7308
This PR addresses needed documentation requested in several help desk tickets. This is connected to GitHub issues:
#8404
#8306
Checklist for maintainers
CHANGES.rst
within the relevant release sectionHow to run regression tests on a PR