-
Notifications
You must be signed in to change notification settings - Fork 340
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Per sample grad correctness util (#532)
Summary: ## Types of changes - [ ] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [x] Docs change / refactoring / dependency upgrade ## Motivation and Context / Related issue Implementation of the utility described in #484. Refactored the code to avoid code duplicates. ## How Has This Been Tested (if it applies) Added the new utility as a test case for existing tests stored in `tests.grad_samples`. ## Checklist - [x] The documentation is up-to-date with the changes I made. - [x] I have read the **CONTRIBUTING** document and completed the CLA (see **CONTRIBUTING**). - [x] All tests passed, and additional code has been covered with new tests. Pull Request resolved: #532 Reviewed By: karthikprasad Differential Revision: D40797432 Pulled By: ffuuugor fbshipit-source-id: 923009d6f7f6d4c34bce9f4af39945fdf9ff9d57
- Loading branch information
1 parent
1e661e8
commit a0a31ba
Showing
11 changed files
with
661 additions
and
335 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.