Skip to content

Thread-safe garbage collection rebase#1073

Open
MilesCranmer wants to merge 3 commits into
JuliaPy:masterfrom
MilesCranmer:threadsafe_gc_2
Open

Thread-safe garbage collection rebase#1073
MilesCranmer wants to merge 3 commits into
JuliaPy:masterfrom
MilesCranmer:threadsafe_gc_2

Conversation

@MilesCranmer

Copy link
Copy Markdown
Contributor

Rebased version of #883 from @marius311

My one concern is for cases where thread 1 is busy... Does that mean the GC would simply never run?

marius311 and others added 3 commits February 25, 2021 17:25
[Diff since v1.96.3](JuliaPy/PyCall.jl@v1.96.3...v1.96.4)

**Merged pull requests:**
- Rename `distutils.sysconfig` to `sysconfig` (JuliaPy#1068) (@MilesCranmer)
- bump to 1.96.4 (JuliaPy#1070) (@stevengj)
@MilesCranmer MilesCranmer changed the title Threadsafe garbage collection v2 Thread-safe garbage collection rebase Jan 8, 2024
@codecov-commenter

codecov-commenter commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 67.76%. Comparing base (e54c4ee) to head (9e05155).
⚠️ Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
src/PyCall.jl 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1073      +/-   ##
==========================================
+ Coverage   67.75%   67.76%   +0.01%     
==========================================
  Files          20       20              
  Lines        2025     2029       +4     
==========================================
+ Hits         1372     1375       +3     
- Misses        653      654       +1     
Flag Coverage Δ
unittests 67.76% <85.71%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants