-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Duet AI documentation lookup prompts for Beam ML #30351
Duet AI documentation lookup prompts for Beam ML #30351
Conversation
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Daria Bezkorovaina <[email protected]>
…hub.com/akvelon/beam into duet-ai-ml-documentation-lookup-prompts
How can I use Apache Beam in my AI/ML pipeline? | ||
|
||
Response: | ||
You can use Apache Beam for data ingestion, validation, preprocessing, and inference. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use Apache Beam for data ingestion, validation, preprocessing, and inference. | |
You can use Apache Beam for various tasks within your AI/ML pipeline, including data validation, preprocessing, model validation, and model deployment and inference. |
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup/36_ai_kfp_orchestration.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Daria Bezkorovaina <[email protected]>
Co-authored-by: Daria Bezkorovaina <[email protected]>
Co-authored-by: Daria Bezkorovaina <[email protected]>
Co-authored-by: Daria Bezkorovaina <[email protected]>
@dariabezkorovaina all your comments are resolved |
Assigning reviewers. If you would like to opt out of this review, comment R: @jrmccluskey added as fallback since no labels match configuration Available commands:
The PR bot will only process comments in the main thread (not review comments). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nits mostly related to line spacing and formatting of the section names after removing links from them:
- Used single quotation marks ' ' for section names
- Used
- Removed
learning/prompts/documentation-lookup-nolinks/33_ai_ml_pipelines.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/33_ai_ml_pipelines.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/33_ai_ml_pipelines.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/33_ai_ml_pipelines.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/38_ai_runinference.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Daria Bezkorovaina <[email protected]>
Co-authored-by: Daria Bezkorovaina <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Hey @damccorm, this one is also ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, had a few comments - the LLM piece is the only really significant one. Please make edits in both the no link and with links sections!
Thanks!
learning/prompts/documentation-lookup-nolinks/38_ai_runinference.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/40_ai_bigquery_ml.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/42_ai_custom_inference.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup-nolinks/43_ai_llm_inference.md
Outdated
Show resolved
Hide resolved
Thank you @damccorm Done!. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just had one more minor nit
learning/prompts/documentation-lookup-nolinks/42_ai_custom_inference.md
Outdated
Show resolved
Hide resolved
learning/prompts/documentation-lookup/42_ai_custom_inference.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Danny McCormick <[email protected]>
Thank you @damccorm! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
* ml prompts initial commit * preprocess data added * orchedstration prompts added * orchestration prompts * new files added * ml prompts added * bigqueryml prompt added * fix whitespace * add model refresh prompt * custom inference added * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * llm inference added * whitespace fixed * code review comments * nolinks docs added * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * fixed broken link * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * Apply suggestions from code review Co-authored-by: Daria Bezkorovaina <[email protected]> * code review comments * whitespace fix * Apply suggestions from code review Co-authored-by: Danny McCormick <[email protected]> --------- Co-authored-by: Daria Bezkorovaina <[email protected]> Co-authored-by: Danny McCormick <[email protected]>
First part of beam ml lookup prompts
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
addresses #123
), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, commentfixes #<ISSUE NUMBER>
instead.CHANGES.md
with noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.