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
List backends dynamically based on availability #697
Conversation
Codecov Report
@@ Coverage Diff @@
## master #697 +/- ##
==========================================
- Coverage 89.29% 89.24% -0.05%
==========================================
Files 23 23
Lines 2335 2343 +8
==========================================
+ Hits 2085 2091 +6
- Misses 250 252 +2
Continue to review full report at Codecov.
|
8daca0d
to
a18ed45
Compare
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
_BACKEND = 'soundfile' | ||
if _mod_utils.is_module_available('torchaudio._torchaudio'): | ||
_BACKENDS['sox'] = _sox_backend | ||
_BACKEND = 'sox' |
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.
nit: here, "sox" would become the default since it's the last one selected
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.
Yes, that's the intension but it was a bit unusual so I updated to use regular if 'sox' else if 'soundfile'
, which is easier to read.
Thanks! |
* Update C++ DCGAN example for v1.4.0 * fix Reduction::Sum * FeatureDropout -> Dropout2d
No description provided.