Skip to content

Preserve metric construction in offloaded clones (#4582) - #4582

Open
jeffkbkim wants to merge 1 commit into
meta-pytorch:mainfrom
jeffkbkim:export-D115919787
Open

Preserve metric construction in offloaded clones (#4582)#4582
jeffkbkim wants to merge 1 commit into
meta-pytorch:mainfrom
jeffkbkim:export-D115919787

Conversation

@jeffkbkim

@jeffkbkim jeffkbkim commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary:

Preserve metric-specific construction semantics when creating an offloaded reporting copy. This prevents valid metric options from silently reverting to defaults.

Reviewed By: TroyGarden

Differential Revision: D115919787

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 17, 2026
@meta-codesync

meta-codesync Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

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

@meta-codesync meta-codesync Bot changed the title Preserve metric construction in offloaded clones Preserve metric construction in offloaded clones (#4582) Aug 18, 2026
jeffkbkim added a commit to jeffkbkim/torchrec that referenced this pull request Aug 18, 2026
Summary:

Preserve metric-specific construction semantics when creating an offloaded reporting copy. This prevents valid metric options from silently reverting to defaults.

Differential Revision: D115919787
Summary:

Preserve metric-specific construction semantics when creating an offloaded reporting copy. This prevents valid metric options from silently reverting to defaults.

Reviewed By: TroyGarden

Differential Revision: D115919787
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 Facebook bot. Authors need to sign the CLA before a PR can be reviewed. meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant