Skip to content

{Tests-WIP}[Torchrec] Add context manager to use next batch context for postprocs #2939

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

che-sh
Copy link
Contributor

@che-sh che-sh commented May 3, 2025

Summary: Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600

@facebook-github-bot facebook-github-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 May 3, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73824600

che-sh added a commit to che-sh/torchrec that referenced this pull request May 3, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
che-sh added a commit to che-sh/torchrec that referenced this pull request May 3, 2025
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
che-sh added a commit to che-sh/torchrec that referenced this pull request May 5, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
che-sh added a commit to che-sh/torchrec that referenced this pull request May 5, 2025
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@che-sh che-sh force-pushed the export-D73824600 branch from da3104a to 89073a0 Compare May 8, 2025 01:37
che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73824600

che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@che-sh che-sh force-pushed the export-D73824600 branch from 89073a0 to 4fb1a55 Compare May 8, 2025 01:44
che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@che-sh che-sh force-pushed the export-D73824600 branch from 4fb1a55 to 2e2ab40 Compare May 8, 2025 06:33
che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73824600

che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@che-sh che-sh force-pushed the export-D73824600 branch from 2e2ab40 to 9a0bfa5 Compare May 8, 2025 06:36
che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@che-sh che-sh force-pushed the export-D73824600 branch from 9a0bfa5 to 291f0f2 Compare May 8, 2025 08:03
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73824600

che-sh added a commit to che-sh/torchrec that referenced this pull request May 8, 2025
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Differential Revision: D73824600
@che-sh che-sh force-pushed the export-D73824600 branch 2 times, most recently from 5391139 to c21b926 Compare May 15, 2025 06:30
che-sh added a commit to che-sh/torchrec that referenced this pull request May 15, 2025
…2939)

Summary:

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Reviewed By: TroyGarden

Differential Revision: D73824600
…2939)

Summary:
Pull Request resolved: pytorch#2939

Small refactor to reduce code repetition of setting and reverting pipelined postprocs context to the next batch's context

Reviewed By: TroyGarden

Differential Revision: D73824600
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73824600

@che-sh che-sh force-pushed the export-D73824600 branch from c21b926 to 40ab70d Compare May 15, 2025 06:33
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. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants