Feature: Google Drive - Presentation to Markdown / Audio|Video Transcribed (10MB Limit) #3075
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…#2692) Co-authored-by: Sean Story <[email protected]>
…e id and adding support for views (elastic#2681) (elastic#2688) Co-authored-by: parthpuri-elastic <[email protected]>
…tic#2699) (elastic#2700) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
elastic#2713) Co-authored-by: Liam Thompson <[email protected]>
…elastic#2715) Co-authored-by: Artem Shelkovnikov <[email protected]>
…tic#2721) (elastic#2724) Co-authored-by: elastic-renovate-prod[bot] <174716857+elastic-renovate-prod[bot]@users.noreply.github.com> Co-authored-by: Giannicola Olivadoti <[email protected]> Co-authored-by: Artem Shelkovnikov <[email protected]>
Co-authored-by: Artem Shelkovnikov <[email protected]>
…lastic#2737) Co-authored-by: parthpuri-elastic <[email protected]>
Co-authored-by: Sean Story <[email protected]>
) Co-authored-by: Artem Shelkovnikov <[email protected]>
…tic#2856) Co-authored-by: Sean Story <[email protected]>
elastic#2871) Co-authored-by: Sean Story <[email protected]>
…elastic#2881) (elastic#2900) Co-authored-by: Artem Shelkovnikov <[email protected]>
…) (elastic#2973) Co-authored-by: Sean Story <[email protected]> Co-authored-by: Artem Shelkovnikov <[email protected]>
…nner (elastic#2984) (elastic#2988) Co-authored-by: Jedr Blaszyk <[email protected]>
Added the ability to send presentations to azure openai gpt-4o-mini for conversation to markdown Added the ability for audio/video files under 25MB to be sent to azure open whisper model for output to text.
seanstory
requested changes
Jan 6, 2025
There was a problem hiding this comment.
Looks like you may need to rebase and force-push - you're pulling in a lot of diff that is not yours.
Also, it looks like you're tightly coupling the Google Drive connector to an Azure AI tool. Which doesn't make a lot of sense to me.
I don't see this being mergable. If you'd like to have a chat with the team in #connectors-feedback, we can discuss how we can work to better integrate with LLM tools, but this diff as I understand it does not align with our vision for the architecture.
artem-shelkovnikov
requested changes
Jan 7, 2025
|
Thanks for the review, and I agree with all points. I am glad to see that this is kicking a more considerable discussion because markdown and text are the future for any GenAI use cases. 😄 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #3073 & #3074
In the Google Drive source this adds the functionality to convert Presentation to Markdown using Azure's OpenAI models and using Whisper service to transcribe audio or video to text with the limit of 10MB.
Checklists
Pre-Review Checklist
config.yml.example)v7.13.2,v7.14.0,v8.0.0)Changes Requiring Extra Attention
This adds external services for Azure hosted OpenAI GPT4-mini and Whisper API's
Related Pull Requests
Release Note