-
Notifications
You must be signed in to change notification settings - Fork 383
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
Make cross-validation test less repetitive and add device and dtype checks #2268
Conversation
This pull request was exported from Phabricator. Differential Revision: D55572962 |
…hecks (pytorch#2268) Summary: - Deduplicated logic between the cases with and without noise observations - Broke up into subtests - Added checks for Yvar produced by cross-validation - Added checks for device and dtype of results Differential Revision: D55572962
…hecks (pytorch#2268) Summary: - Deduplicated logic between the cases with and without noise observations - Broke up into subtests - Added checks for Yvar produced by cross-validation - Added checks for device and dtype of results Differential Revision: D55572962
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2268 +/- ##
=======================================
Coverage 99.97% 99.97%
=======================================
Files 197 197
Lines 17148 17148
=======================================
Hits 17144 17144
Misses 4 4 ☔ View full report in Codecov by Sentry. |
…hecks (pytorch#2268) Summary: - Deduplicated logic between the cases with and without noise observations - Broke up into subtests - Added checks for Yvar produced by cross-validation - Added checks for device and dtype of results Reviewed By: Balandat Differential Revision: D55572962
50b7de3
to
d42a982
Compare
This pull request was exported from Phabricator. Differential Revision: D55572962 |
…hecks (pytorch#2268) Summary: - Deduplicated logic between the cases with and without noise observations - Broke up into subtests - Added checks for Yvar produced by cross-validation - Added checks for device and dtype of results Reviewed By: Balandat Differential Revision: D55572962
d42a982
to
78290ca
Compare
This pull request was exported from Phabricator. Differential Revision: D55572962 |
This pull request has been merged in ad848f6. |
Summary:
Differential Revision: D55572962