-
Notifications
You must be signed in to change notification settings - Fork 70
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Summary: Pull Request resolved: #164 Looking for some feedback on this approach. The scheduler API has 40 methods, and trying to test each one individually would be overkill I think. Plus, the implementation for a lot of the gates is very similar. I figured the best thing to do would be to just create a bunch of dependent gates, that way we can test that the lazy scheduler properly batches things together more efficiently than the eager scheduler. If anyone has suggestions for more thorough things to test, LMK. Reviewed By: RuiyuZhu Differential Revision: D35162133 fbshipit-source-id: 3d2cdfa5b01a2cd2a321c471eacc0377a8650ba4
- Loading branch information
1 parent
192d4b9
commit c7711fd
Showing
1 changed file
with
70 additions
and
0 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