Skip to content

Conversation

@mthrok
Copy link
Contributor

@mthrok mthrok commented Jan 21, 2021

  • Do not install system-wide sox, so that it would not get mixed up with static one torchaudio builds
  • Do not install system-wide cmake. It's installed at test.
  • Move libsndfile installation to Docker image

* Do not install system-wide sox, so that it would not get mixed up with static one torchaudio builds
* Move libsndfile installation to Docker image
@mthrok mthrok marked this pull request as ready for review January 21, 2021 19:55
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

Stamp!

@mthrok mthrok merged commit 5547f20 into pytorch:master Jan 21, 2021
@mthrok mthrok deleted the docker branch January 21, 2021 20:00
mthrok pushed a commit to mthrok/audio that referenced this pull request Feb 26, 2021
* [iOS][GPU] Add iOS GPU workflow (pytorch#1200)

* pt mobile script and optimize recipe (pytorch#1193)

* pt mobile script and optimize recipe

* 1 pt mobile new recipes summary and 5 recipes

* updated recipes_index.rst

* thumbnail png fix for ios recipe in recipes_index.rst

* edits based on feedback

* Updating 1.7 branch (pytorch#1205)

* Update event tracking (pytorch#1188)

* Update beginner_source/audio_preprocessing_tutorial.py (pytorch#1199)

* Typo in beginner_source/audio_preprocessing_tutorial.py

Typo in beginner_source/audio_preprocessing_tutorial.py

fron > from

* update title.

* fix file access.

Co-authored-by: JuHyuk Park <[email protected]>

* Update audio_preprocessing_tutorial.py (pytorch#1202)

Adds a comment for running this tutorial in Google Colab.

Co-authored-by: Pat Mellon <[email protected]>
Co-authored-by: Vincent QB <[email protected]>
Co-authored-by: JuHyuk Park <[email protected]>

Co-authored-by: Tao Xu <[email protected]>
Co-authored-by: Jeff Tang <[email protected]>
Co-authored-by: Pat Mellon <[email protected]>
Co-authored-by: Vincent QB <[email protected]>
Co-authored-by: JuHyuk Park <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants