Skip to content

[wip] shared memory optimization for policy weight update#402

Closed
casteryh wants to merge 34 commits intomainfrom
yhu/shared-tensor
Closed

[wip] shared memory optimization for policy weight update#402
casteryh wants to merge 34 commits intomainfrom
yhu/shared-tensor

Conversation

@casteryh
Copy link
Copy Markdown
Contributor

No description provided.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 14, 2025
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.04110% with 48 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@399b20d). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/forge/actors/policy.py 2.94% 33 Missing ⚠️
tests/unit_tests/util/test_shared_tensor.py 97.02% 9 Missing ⚠️
src/forge/util/_shared_tensor.py 94.05% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #402   +/-   ##
=======================================
  Coverage        ?   66.06%           
=======================================
  Files           ?       81           
  Lines           ?     8127           
  Branches        ?        0           
=======================================
  Hits            ?     5369           
  Misses          ?     2758           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 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.

@pbontrager pbontrager requested a review from LucasLLC October 15, 2025 14:59
@casteryh casteryh closed this Oct 15, 2025
@casteryh casteryh deleted the yhu/shared-tensor branch October 16, 2025 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants