-
Notifications
You must be signed in to change notification settings - Fork 225
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
Support audio duration mismatch tolerance in MixedCut.load_audio()
#1054
Conversation
return mixture | ||
|
||
|
||
def _create_mixture( |
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.
Is there a reason why this function is moved outside the class? The algorithm is specific to this particular simulator.
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.
Makes sense, I changed this back and used the same "trick" as in the simulator.
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.
Thanks. Could you do the same for _simulate_worker
(i.e. make it a function of the specific class)? The logic for those is also tied to that algorithm.
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.
That would complicate the workarounds for parallelization; anyway each simulation method has a separate file, so I think it should be OK and actually consistent with other functions in lhotse that are being parallelized this way, e.g. _X_single()
functions at the end of lhotse/cut/set.py
.
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.
Okay, LGTM.
No description provided.