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 option to pass worker url to the ffmpeg.load #639

Closed
wants to merge 1 commit into from

Conversation

kisel4363
Copy link

No description provided.

Copy link

netlify bot commented Dec 3, 2023

Deploy Preview for ffmpegwasm canceled.

Name Link
🔨 Latest commit 9c6620a
🔍 Latest deploy log https://app.netlify.com/sites/ffmpegwasm/deploys/656d0187dd6b6e000897003c

@PaulieScanlon
Copy link

🥳

@jeromewu
Copy link
Collaborator

@kisel4363 I think this PR doesn't make sense as the workerURL in the config is for ffmpeg-core.worker.js, so passing the worker here doesn't work.

Can you share more context of your use case here?

@jeromewu jeromewu added the invalid This doesn't seem right label Dec 17, 2023
@jeromewu jeromewu mentioned this pull request Dec 17, 2023
@PaulieScanlon
Copy link

Hey @jeromewu. Here's the issue I opened in the Remix repo.

remix-run/remix#8215

Not sure if this is a Remix issue or an ffmpeg/WASM issue.

Would you be able to take a look at the reproduction repo I created (linked in the above issue)... I'm kinda stuck.

@jeromewu
Copy link
Collaborator

I have no experience in remix and thus don't know how to help. You can try 0.12.8 and see if assigning a custom classWorkerURL helps.

@dchristensen
Copy link
Contributor

@jeromewu The fix from #652 does not seem to be in the ESM module for 0.12.8:
https://unpkg.com/browse/@ffmpeg/[email protected]/dist/esm/classes.js#L102

@jeromewu
Copy link
Collaborator

@dchristensen Fixed in 0.12.9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants