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

Add jitscriped to_dict to KJT benchmark #1871

Closed
wants to merge 2 commits into from

Conversation

gnahzg
Copy link
Contributor

@gnahzg gnahzg commented Apr 11, 2024

Summary: As titled

Differential Revision: D56033283

@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 Apr 11, 2024
@facebook-github-bot
Copy link
Contributor

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

gnahzg added a commit to gnahzg/torchrec that referenced this pull request Apr 12, 2024
Summary:

As titled

Differential Revision: D56033283
@gnahzg gnahzg force-pushed the export-D56033283 branch from 8600bee to de6851f Compare April 12, 2024 10:57
@facebook-github-bot
Copy link
Contributor

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

gnahzg added 2 commits April 16, 2024 11:45
Summary:

As initial support for KJT benchmarking, currently only benchmark to_dict function for performance

TODO:
Benchmark more Kjt functions
Add functionality to support torchscripted KJT functions
Support memory profiling

Differential Revision: D55324788
Summary:

As titled

Differential Revision: D56033283
@gnahzg gnahzg force-pushed the export-D56033283 branch from de6851f to 8c1643e Compare April 16, 2024 18:45
@facebook-github-bot
Copy link
Contributor

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

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