Skip to content

Enable CommsMonitor by default#997

Closed
YulunW wants to merge 1 commit intometa-pytorch:mainfrom
YulunW:export-D95753544
Closed

Enable CommsMonitor by default#997
YulunW wants to merge 1 commit intometa-pytorch:mainfrom
YulunW:export-D95753544

Conversation

@YulunW
Copy link
Contributor

@YulunW YulunW commented Mar 9, 2026

Summary: CommsMonitor is a component that holds the tracing + metadata for each communicator. It is meant to be used so that communicator information can be safely dumped even after the destruction of communicator is called. We wanted to also use it do memory logging, thus enable it by default.

Differential Revision: D95753544

Summary: CommsMonitor is a component that holds the tracing + metadata for each communicator. It is meant to be used so that communicator information can be safely dumped even after the destruction of communicator is called. We wanted to also use it do memory logging, thus enable it by default.

Differential Revision: D95753544
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Mar 9, 2026
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Mar 9, 2026

@YulunW has exported this pull request. If you are a Meta employee, you can view the originating Diff in D95753544.

@meta-codesync
Copy link
Contributor

meta-codesync bot commented Mar 9, 2026

This pull request has been merged in 90d16c9.

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. fb-exported Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant