Piotr Żelasko
Piotr Żelasko
You can call .to_eager() on the resulting manifest to evaluate the expression. I started moving multiple operations to lazy evaluation a while ago, the only downside I can see is...
Hmm yeah that code was written before I got the idea with lazy manifests. We’d need to adapt it somehow or make some checks optional and emit warnings if it’s...
we can either 1) replace some integrity checks/fixes like `remove_missing_recordings_and_supervisions` with a two-pass strategy that first iterates to get the sets of compatible/incompatible IDs, and in the second pass filters...
Yeah, I guess you're right. I probably would leave it as-is for now, unless you'd like to improve it further.
Good point. This function is actually not very well supported in Lhotse right now -- please refer to the discussion in #532. If you're open to doing some work to...
Hmm it looks like that code has to iterate over 39.6M cuts to reach the point at which your training checkpoint was saved. It makes sense to me that it...
LGTM. Can you first fix the formatting issues? Don't worry about unit test, it was a randomness-related error.
As far as I remember, the unit tests failed on some test that used random numbers and can crash very rarely; I will fix that separately, some time. Can you...
Thanks, can you also merge master and resolve the conflicts?