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

Agent Build and Deploy - New #157

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kanchpat
Copy link
Contributor

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

  • Contribution Guidelines: I have read the Contribution Guidelines.
  • CLA: I have signed the CLA.
  • Authorship: I am listed as the author (if applicable).
  • Conventional Commits: My PR title and commit messages follow the Conventional Commits spec.
  • Code Format: I have run nox -s format to format the code.
  • Spelling: I have fixed any spelling errors, and added false positives to .github/actions/spelling/allow.txt if necessary.
  • Template: I have followed the aaie_notebook_template.ipynb if submitting a new jupyter notbook.
  • Sync: My Fork is synced with the upstream.
  • Documentations: I have updated relevant documentations (if applicable) in the docs folder.

Function Calling , Vertex AI Search, Vertex Extensions
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

@code-review-assist code-review-assist bot left a 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

Copy link

@code-review-assist code-review-assist bot left a 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 @@
{
Copy link
Collaborator

@RajeshThallam RajeshThallam Dec 20, 2024

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 @@
{
Copy link
Collaborator

@RajeshThallam RajeshThallam Dec 20, 2024

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 @@
{
Copy link
Collaborator

@RajeshThallam RajeshThallam Dec 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you provide link to files that user can upload that goes with the prompt?


Reply via ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants