Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a new Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #37917 +/- ##
==========================================
Coverage 54.75% 54.75%
Complexity 1690 1690
==========================================
Files 1063 1064 +1
Lines 166898 167111 +213
Branches 1226 1226
==========================================
+ Hits 91377 91505 +128
- Misses 73328 73413 +85
Partials 2193 2193
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces the ADKAgentModelHandler to enable running Google Agent Development Kit (ADK) agents within Apache Beam's RunInference transform. This includes a new Python module for the handler, comprehensive unit tests, and updates to setup.py to include the google-adk dependency. Feedback suggests improving performance by using asyncio.gather for concurrent agent invocations instead of repeated asyncio.run calls, and addressing an import consistency issue in the test file.
sdks/python/apache_beam/ml/inference/agent_development_kit_test.py
Outdated
Show resolved
Hide resolved
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces the ADKAgentModelHandler to enable running Google Agent Development Kit (ADK) agents within Apache Beam's RunInference transform. This includes the core implementation, a dedicated test file, and updates to dependencies and the CHANGES.md file. Feedback indicates a critical bug where model.session_service.create_session will raise a SessionExistsError for elements beyond the first when a shared session_id is used, requiring a try...except block. Additionally, the response extraction in _invoke_agent should be improved to handle multi-part agent responses completely by using event.content.text instead of event.content.parts[0].text.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces the ADKAgentModelHandler to integrate Google Agent Development Kit (ADK) agents with Apache Beam's RunInference transform, allowing users to run ADK agents within Beam pipelines. This involves adding a new Python module, corresponding unit tests, updating the CHANGES.md file, and adding google-adk as a dependency in setup.py. A review comment highlighted a bug where SessionExistsError is used in the new handler without being imported, which could lead to a NameError.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces ADKAgentModelHandler for running Google Agent Development Kit (ADK) agents within Apache Beam's RunInference transform, along with corresponding unit tests. It also adds google-adk as a dependency. A critical issue was identified where the sessions module was used without being imported, which would lead to a NameError at runtime.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
Really exciting PR. This could be like Flink Agents of Beam. |
Adds model handler capable of running ADK agents mid-pipeline.
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.mdwith 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.