Skip to content

Conversation

mthrok
Copy link
Contributor

@mthrok mthrok commented May 8, 2020

Fixes #613

@mthrok mthrok requested a review from vincentqb May 8, 2020 18:42
Copy link
Contributor

@vincentqb vincentqb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit:flake8

torchaudio/compliance/kaldi.py:93:1: E124 closing bracket does not match visual indentation
torchaudio/compliance/kaldi.py:555:1: W293 blank line contains whitespace

More importantly: not compatible with current windows setup?

Build-agent version 1.0.31788-6ad45373 (2020-05-07T13:35:11+0000)
Creating a dedicated VM with windows-server-2019-vs2019:stable image
failed to create host: Image windows-server-2019-vs2019:stable is not compatible with resource class l1.medium

failed to create host: Image windows-server-2019-vs2019:stable is not compatible with resource class l1.medium

@codecov
Copy link

codecov bot commented May 12, 2020

Codecov Report

Merging #619 into master will increase coverage by 0.02%.
The diff coverage is 86.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #619      +/-   ##
==========================================
+ Coverage   89.00%   89.03%   +0.02%     
==========================================
  Files          21       21              
  Lines        2255     2261       +6     
==========================================
+ Hits         2007     2013       +6     
  Misses        248      248              
Impacted Files Coverage Δ
torchaudio/compliance/kaldi.py 96.05% <86.36%> (+0.07%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00d3820...c2cd73e. Read the comment docs.

@mthrok mthrok requested a review from vincentqb May 12, 2020 03:50
@mthrok
Copy link
Contributor Author

mthrok commented May 12, 2020

More importantly: not compatible with current windows setup?

Build-agent version 1.0.31788-6ad45373 (2020-05-07T13:35:11+0000)
Creating a dedicated VM with windows-server-2019-vs2019:stable image
failed to create host: Image windows-server-2019-vs2019:stable is not compatible with resource class l1.medium

failed to create host: Image windows-server-2019-vs2019:stable is not compatible with resource class l1.medium

It's unrelated.

Copy link
Contributor

@vincentqb vincentqb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vincentqb vincentqb merged commit 3a4f356 into pytorch:master May 12, 2020
@mthrok mthrok deleted the kaldi-fix branch May 13, 2020 02:41
bhargavkathivarapu pushed a commit to bhargavkathivarapu/audio that referenced this pull request May 19, 2020
* Make fbank support cuda

* Reduce rtol for kaldi

* fix test

* fix flake8
mthrok pushed a commit to mthrok/audio that referenced this pull request Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

torchaudio.compliance.kaldi.fbank does NOT support GPU

2 participants