-
Notifications
You must be signed in to change notification settings - Fork 355
Update ADK doc according to issue #744 - 9 #765
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
Conversation
code_executor=gke_executor, | ||
) | ||
``` | ||
|
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.
BREAKING CHANGE: removal of content
```py | ||
--8<-- "examples/python/snippets/tools/built-in-tools/vertexai_rag_engine.py" | ||
``` | ||
|
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.
BREAKING CHANGE: removal of content
```py | ||
--8<-- "examples/python/snippets/tools/built-in-tools/vertexai_search.py" | ||
``` | ||
|
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.
BREAKING CHANGE: removal of content
* **`list_table_ids`**: Fetches table ids present in a BigQuery dataset. | ||
* **`get_table_info`**: Fetches metadata about a BigQuery table. | ||
* **`execute_sql`**: Runs a SQL query in BigQuery and fetch the result. | ||
* **`execute_sql`**: Runs a SQL query in BigQuery and fetch the result. This tool now includes a `dry_run` parameter. If set to `True`, the query will be validated by BigQuery but not actually executed, returning information about the query's validity and potential cost. |
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.
duplicative update
* **`execute_sql`**: Runs a SQL query in BigQuery and fetch the result. This tool now includes a `dry_run` parameter. If set to `True`, the query will be validated by BigQuery but not actually executed, returning information about the query's validity and potential cost. | ||
* **`forecast`**: Runs a BigQuery AI time series forecast using the `AI.FORECAST` function. | ||
* **`ask_data_insights`**: Answers questions about data in BigQuery tables using natural language. | ||
* **`analyze_contribution`**: Runs a BigQuery ML contribution analysis using `ML.CREATE_MODEL` and `ML.GET_INSIGHTS` to identify the dimensions that contribute most significantly to a given metric. |
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.
duplicative update
|
||
search_agent = Agent( | ||
model='gemini-2.0-flash', | ||
model='gemini-1.5-flash', |
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.
incorrect update
) | ||
coding_agent = Agent( | ||
model='gemini-2.0-flash', | ||
model='gemini-1.5-flash', |
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.
incorrect update
root_agent = Agent( | ||
name="RootAgent", | ||
model="gemini-2.0-flash", | ||
model="gemini-1.5-flash", |
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.
incorrect update
instruction=""" | ||
You're a specialist in Code Execution | ||
""", | ||
instruction=""""\n You're a specialist in Code Execution\n """", |
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.
trival, uneeded update.
.subAgents(searchAgent, codingAgent) // Not supported, as the sub agents use built in tools. | ||
.build(); | ||
``` | ||
``` |
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.
BREAKING CHANGE: removal of content
DECISION: REJECT
|
This PR removes the limitation notice on using multiple built-in tools and adds a new section explaining how to use them together with a code example.