-
Notifications
You must be signed in to change notification settings - Fork 229
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
lhotse kaldi import fails silently #719
Comments
it's probably my mess. Sorry about that.
Would you be able to provide repro on the side?
y.
…On Wed, May 18, 2022 at 4:14 PM Nagendra Goel ***@***.***> wrote:
Lhotse Kaldi import used to throw an error if there was something wrong in
the data directory. Now, when I updated to the latest version, it seems to
be failing silently and does not create supervisions (creates recordings).
Is that an intended change in the code? I kind of knew what files to put in
place, but it would be ideal if the code will warn what is missing for
supervisions, or give the option to skip supervisions.
—
Reply to this email directly, view it on GitHub
<#719>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACUKYX2RYCYPE3JBXIQJG2LVKVFTVANCNFSM5WJTYWNA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
it should definitely scream and kick when something aint right
y.
…On Wed, May 18, 2022 at 4:15 PM Jan Yenda Trmal ***@***.***> wrote:
it's probably my mess. Sorry about that.
Would you be able to provide repro on the side?
y.
On Wed, May 18, 2022 at 4:14 PM Nagendra Goel ***@***.***>
wrote:
> Lhotse Kaldi import used to throw an error if there was something wrong
> in the data directory. Now, when I updated to the latest version, it seems
> to be failing silently and does not create supervisions (creates
> recordings). Is that an intended change in the code? I kind of knew what
> files to put in place, but it would be ideal if the code will warn what is
> missing for supervisions, or give the option to skip supervisions.
>
> —
> Reply to this email directly, view it on GitHub
> <#719>, or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ACUKYX2RYCYPE3JBXIQJG2LVKVFTVANCNFSM5WJTYWNA>
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: ***@***.***>
>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Lhotse Kaldi import used to throw an error if there was something wrong in the data directory. Now, when I updated to the latest version, it seems to be failing silently and does not create supervisions (creates recordings). Is that an intended change in the code? I kind of knew what files to put in place, but it would be ideal if the code will warn what is missing for supervisions, or give the option to skip supervisions.
The text was updated successfully, but these errors were encountered: