Skip to content

Conversation

@mthrok
Copy link
Contributor

@mthrok mthrok commented Feb 22, 2023

This commit is kind of clean up and preparation for future
development.

We plan to pass around more complicated objects among
StreamReader and StreamWriter, and TorchBind is not expressive enough
for defining intermediate object, so we use PyBind11 for binding
StreamWriter.

@facebook-github-bot
Copy link
Contributor

@mthrok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@mthrok mthrok force-pushed the update-ffmpeg-binding branch 10 times, most recently from 6f8baab to 7ff7590 Compare February 23, 2023 23:21
@facebook-github-bot
Copy link
Contributor

@mthrok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

1 similar comment
@facebook-github-bot
Copy link
Contributor

@mthrok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@mthrok mthrok force-pushed the update-ffmpeg-binding branch from 7ff7590 to 4ddee36 Compare February 24, 2023 02:58
@facebook-github-bot
Copy link
Contributor

@mthrok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@mthrok mthrok force-pushed the update-ffmpeg-binding branch from 4ddee36 to b72cda0 Compare February 24, 2023 03:50
@mthrok mthrok changed the title Bind StreamWriter with PyBind11 Bind StreamReader/Writer with PyBind11 Feb 24, 2023
@facebook-github-bot
Copy link
Contributor

@mthrok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@mthrok mthrok force-pushed the update-ffmpeg-binding branch from b72cda0 to effd933 Compare February 24, 2023 04:30
@facebook-github-bot
Copy link
Contributor

@mthrok has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D43515714

mthrok added a commit to mthrok/audio that referenced this pull request Feb 24, 2023
Summary:
This commit is kind of clean up and preparation for future
development.

We plan to pass around more complicated objects among
StreamReader and StreamWriter, and TorchBind is not expressive enough
for defining intermediate object, so we use PyBind11 for binding
StreamWriter.

Pull Request resolved: pytorch#3091

Differential Revision: D43515714

Pulled By: mthrok

fbshipit-source-id: 28f5a02eedc58a2754fe8e938619569b26df14ee
@mthrok mthrok force-pushed the update-ffmpeg-binding branch from effd933 to a366a11 Compare February 24, 2023 13:16
Summary:
This commit is kind of clean up and preparation for future
development.

We plan to pass around more complicated objects among
StreamReader and StreamWriter, and TorchBind is not expressive enough
for defining intermediate object, so we use PyBind11 for binding
StreamWriter.

Pull Request resolved: pytorch#3091

Differential Revision: D43515714

Pulled By: mthrok

fbshipit-source-id: cd46cd846320190fcb574c1f05ac5eff0bf2f58e
@mthrok mthrok force-pushed the update-ffmpeg-binding branch from a366a11 to 4c1d44b Compare February 24, 2023 14:08
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D43515714

@facebook-github-bot
Copy link
Contributor

@mthrok merged this pull request in b012b45.

@github-actions
Copy link

Hey @mthrok.
You merged this PR, but labels were not properly added. Please add a primary and secondary label (See https://github.com/pytorch/audio/blob/main/.github/process_commit.py)

@mthrok mthrok deleted the update-ffmpeg-binding branch February 24, 2023 17:04
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.

2 participants