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
Move I/O related Tensor ops to misc_ops module #694
Conversation
Codecov Report
@@ Coverage Diff @@
## master #694 +/- ##
==========================================
+ Coverage 89.19% 89.29% +0.10%
==========================================
Files 22 23 +1
Lines 2341 2335 -6
==========================================
- Hits 2088 2085 -3
+ Misses 253 250 -3
Continue to review full report at Codecov.
|
4accd98
to
3d3e92d
Compare
* also deletes duplicated func
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
@@ -0,0 +1,30 @@ | |||
from typing import Union, Callable |
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: That's a very generic name for the file :)
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.
Yeah, I do not know what to call, but this is just evicting these functions from __init__.py
so if this module glows into something then we can have a better naming.
Thanks! |
…onduct_file_exists Adding Code of Conduct file
check_input
andaudio_normalization
functions defined in__init__.py
,_backend.py
and_soundfilebackend
(duplicated) are referred from multiple modules and cause cyclic module dependencies and preventing modularizing backends. This PR resolve this by putting them in a separate dedicated module_internal.misc_ops
.Also change the helper function name from
audio_normalization
(noun) tonormalize_audio
(verb) for consistency.