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 conprehensive test cases for TestCUDA #133

Closed
wants to merge 3 commits into from
Closed

Add conprehensive test cases for TestCUDA #133

wants to merge 3 commits into from

Conversation

jeffreysijuntan
Copy link
Contributor

Summary: Add comprehensive test case for TestCUDA that would enable us to check the correctness of cuda integration

Differential Revision: D21911937

jeffreysijuntan added 3 commits June 5, 2020 14:16
Summary:
Pull Request resolved: #131

This diff does the two following thing:

1. Implement a encoder for LongTensor that embed fixed-point arithmetic into floating-point arithmetic. It enables us to utilize existing CUDA kernel for floating points to compute matrix multiplication for LongTensor.

2.  Add a new test file test_cuda.py for future CUDA related testing

The encoder currently only supports matmul. Conv2d will be supported in future diffs.

The API design is based on: D21860688

Differential Revision: D21882865

fbshipit-source-id: c4433b2e515b803965d3504eaab00b4ed7e669da
Summary: Support CUDA for conv1d, conv2d, conv1d_transpose, conv2d_transpose

Differential Revision: D21885098

fbshipit-source-id: 90abae49399f715262a9f2d5abfe5bb99fc9dec6
Summary: Add comprehensive test case for TestCUDA that would enable us to check the correctness of cuda integration

Differential Revision: D21911937

fbshipit-source-id: bc22d1f5fa4e24b48b74f87bacc7e88d69ff7acb
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Jun 5, 2020
@facebook-github-bot
Copy link
Contributor

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

@knottb knottb closed this Jun 30, 2020
tanjuntao pushed a commit to tanjuntao/CrypTen that referenced this pull request Nov 27, 2023
Summary:
Pull Request resolved: fairinternal/CrypTen#133

Update to RTD URL

(Note: this ignores all push blocking failures!)

Reviewed By: vshobha

Differential Revision: D17845454

fbshipit-source-id: 9d03ca59ab7bc36141d5357794716cf4b1fc8e74
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.

3 participants