-
Notifications
You must be signed in to change notification settings - Fork 145
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
Agent Build and Deploy - New #157
base: main
Are you sure you want to change the base?
Conversation
Function Calling , Vertex AI Search, Vertex Extensions
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
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.
Hi team,
I'm currently reviewing this pull request and will have a full review ready in a few minutes. In the meantime, here's a quick summary to help you and other reviewers get up to speed:
This pull request, "Agent Build and Deploy - New", by kanchpat focuses on adding new functionality related to Agent building and deployment. Based on the description, it seems to incorporate features for Function Calling, Vertex AI Search, and Vertex Extensions. Unfortunately, without access to the actual patch files, I can't provide a detailed breakdown of the specific code changes. I'll update this summary once I've reviewed the code changes.
However, I can say with certainty that this PR is likely to improve the overall agent development and deployment process. The addition of Vertex AI Search and Extensions suggests enhanced capabilities and integration with Google Cloud's AI platform.
Here's a little something to brighten your day:
Code flows like a river,
Testing's the ocean's vastness,
Bugs, the hidden reefs.
I'll post my complete review shortly.
Best,
Gemini
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.
The pull request title could be more descriptive. Instead of "Agent Build and Deploy - New", consider something like "Add examples for Function Calling, Vertex AI Search, and Vertex Extensions".
The pull request description is too generic. It needs to clearly articulate the changes introduced. Please replace the placeholder text with specific details about the added functionalities for Function Calling, Vertex AI Search, and Vertex AI Extensions. Provide example use cases and explain how these changes benefit users. Be sure to reference any related issues (e.g., "Fixes #123") and list the modified files.
The checklist in the description should be reviewed and updated. Ensure each item is accurately checked or unchecked to reflect the status of the pull request.
Since this pull request introduces examples for Agents, consider adding a dedicated section to the main README.md file to highlight this new content. This will improve discoverability and provide context for users exploring the repository.
Below are specific suggestions for the README.md file to incorporate the new Agent examples and enhance the overall documentation.
@@ -0,0 +1,851 @@ | |||
{ |
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.
Please follow the notebook template (go/aaie-notebook-template) and add links to open notebook on different surfaces.
Reply via ReviewNB
@@ -0,0 +1,851 @@ | |||
{ |
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.
Can you move this section to the top after the title telling what this notebook/tutorial covers followed by existing content?
Reply via ReviewNB
@@ -0,0 +1,851 @@ | |||
{ |
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.
Function Calling , Vertex AI Search, Vertex Extensions
Pull-Request Template
Thank you for your contribution! Please provide a brief description of your changes and ensure you've completed the checklist below.
Description
What does this PR do? Why is it necessary?
Fixes # (if applicable)
Checklist
nox -s format
to format the code.aaie_notebook_template.ipynb
if submitting a new jupyter notbook.