-
Notifications
You must be signed in to change notification settings - Fork 7k
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 tests for LSUN #3454
add tests for LSUN #3454
Conversation
# Currently datasets.LSUN caches the keys in the current directory rather than in the root directory. Thus, | ||
# this creates a number of unique _cache_* files in the current directory that will not be removed together | ||
# with the temporary directory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll send a follow-up PR to fix this.
Codecov Report
@@ Coverage Diff @@
## master #3454 +/- ##
==========================================
- Coverage 76.13% 76.11% -0.02%
==========================================
Files 105 105
Lines 9703 9697 -6
Branches 1556 1556
==========================================
- Hits 7387 7381 -6
Misses 1836 1836
Partials 480 480
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Summary: Reviewed By: fmassa Differential Revision: D26756274 fbshipit-source-id: b489819c79dfb03393a7ee9c2638f9a5bc35c11e Co-authored-by: vfdev <vfdev.5@gmail.com> Co-authored-by: Francisco Massa <fvsmassa@gmail.com>
No description provided.