Skip to content

Update scalable_ccd.cmake - #261

Merged
zfergus merged 1 commit into
mainfrom
bugfix/scalable-ccd-memory-leak
Sep 10, 2026
Merged

zfergus merged 1 commit into
mainfrom
bugfix/scalable-ccd-memory-leak

Conversation

@zfergus

@zfergus zfergus commented Sep 10, 2026

Copy link
Copy Markdown
Member

Description

Update Scalable CCD to fix a memory leak in CUDA.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.75%. Comparing base (929086d) to head (afdf254).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #261   +/-   ##
=======================================
  Coverage   96.74%   96.75%           
=======================================
  Files         191      191           
  Lines       17292    17292           
  Branches      933      933           
=======================================
+ Hits        16730    16731    +1     
+ Misses        562      561    -1     
Flag Coverage Δ
unittests 96.75% <ø> (+<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.

@zfergus
zfergus merged commit c6d7090 into main Sep 10, 2026
14 checks passed
@zfergus
zfergus deleted the bugfix/scalable-ccd-memory-leak branch September 10, 2026 15:22
@zfergus zfergus added this to the v1.6.1 milestone Sep 11, 2026
@zfergus zfergus added the bugfix label Sep 11, 2026
fsichetti added a commit to fsichetti/ipc-toolkit that referenced this pull request Sep 22, 2026
Brings in the GPU LBVH broad phase (ipc-sim#260), its profiling (ipc-sim#263), a
scalable_ccd pin bump (ipc-sim#261), and a docs/output fix (ipc-sim#264).

One conflict, in BroadPhase::detect_collision_candidates: upstream documented
that the candidates are cleared first, while this branch had added the
all_types parameter that ESP contact needs. The merged implementation already
contains both -- candidates.clear() and the all_types branch -- so the
declaration keeps our signature with upstream's wording, and documents
all_types, which had no @PARAM entry before.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant