Skip to content
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

Add ogg/vorbis support to binary distribution #750

Merged
merged 1 commit into from Jun 26, 2020

Conversation

mthrok
Copy link
Collaborator

@mthrok mthrok commented Jun 26, 2020

This PR adds ogg/vorbis support to binary build.

@seemethere Can we slip this into the next release?

Solves #674

@codecov
Copy link

codecov bot commented Jun 26, 2020

Codecov Report

Merging #750 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #750   +/-   ##
=======================================
  Coverage   89.24%   89.24%           
=======================================
  Files          32       32           
  Lines        2519     2519           
=======================================
  Hits         2248     2248           
  Misses        271      271           

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 793eeab...42dd9c2. Read the comment docs.

@mthrok mthrok changed the title Ogg vorbis support to binary build Add ogg/vorbis support to binary distribution Jun 26, 2020
@mthrok mthrok marked this pull request as ready for review June 26, 2020 14:12
@vincentqb
Copy link
Contributor

This PR adds ogg/vorbis support to binary build.

Should we have a test for this?

@seemethere Can we slip this into the next release?

As far as I know, the release is still a few weeks away :)

@mthrok
Copy link
Collaborator Author

mthrok commented Jun 26, 2020

Should we have a test for this?

Test is already there.

@mthrok mthrok mentioned this pull request Jun 26, 2020
6 tasks
@vincentqb
Copy link
Contributor

Should we have a test for this?

Test is already there.

Right :)

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

@mthrok mthrok merged commit 4daf2fb into pytorch:master Jun 26, 2020
@mthrok mthrok deleted the ogg-vorbis branch June 26, 2020 17:19
@mthrok
Copy link
Collaborator Author

mthrok commented Jun 26, 2020

Thanks!

mthrok added a commit to mthrok/audio that referenced this pull request Jun 29, 2020
mthrok added a commit to mthrok/audio that referenced this pull request Jun 29, 2020
mthrok added a commit that referenced this pull request Jun 29, 2020
* Add vorbis to binary build (#750)

* Fix lint error
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.

None yet

2 participants