-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
Support Video Proxy and TED Embedding #10819
Conversation
@hjlarry Can you review this as well? |
Hi @taowang1993 I tried many times with different model and prompt, but I can't make the video display as expected. This is my screenshot and log:
|
api/core/tools/provider/builtin/duckduckgo/tools/ddgo_video.yaml
Outdated
Show resolved
Hide resolved
The video embedding feature will only supported when Dify supports iframe embedding.
Once Dify allows iframe, this feature will work. Regarding the duplicates and the URL format explainer, I will fix them in a moment. |
@hjlarry I just fixed the duplicates and proxy url explainer. Regarding the ways of presenting the videos , if developers do not want to embed the video, they need to explicitly say it in the system prompt. If they want to embed the video, they will have to either:
|
LGTM |
Summary
This PR adds two features to the DuckDuckGo Video Search tool.
Feature 1: Video Proxy
Users can input a video proxy url.
This feature will prepend the proxy url to the video url.
For example: https://proxy.com/https://youtube.com
Screenshots