Skip to content
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

[torchcodec] Revert register_fake back to imply_abstract #69

Closed
wants to merge 1 commit into from

Conversation

kaiyuey
Copy link

@kaiyuey kaiyuey commented Jul 9, 2024

Summary:
register_fake is only available after Pytorch 2.4, which will be available in late July. We don't want to break user experience if they on old pytorch versions. It is also possible when we release torchcodec, pytorch 2.4 not yet released.

Note: it's expected that reverting back will trigger a lot of logging.

Differential Revision: D59018191

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 9, 2024
@facebook-github-bot
Copy link
Contributor

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

Summary:
Pull Request resolved: #69

`register_fake` is only available after Pytorch 2.4, which will be available in late July. We don't want to break user experience if they on old pytorch versions. It is also possible when we release torchcodec, pytorch 2.4 not yet released.

Note: it's expected that reverting back will trigger a lot of logging.

Reviewed By: NicolasHug

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 007d9b4.

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants