Skip to content

feat: Gemini 3.1 Flash Live#5

Merged
thorwebdev merged 6 commits intomainfrom
thor/gemini-3-eap
Mar 26, 2026
Merged

feat: Gemini 3.1 Flash Live#5
thorwebdev merged 6 commits intomainfrom
thor/gemini-3-eap

Conversation

@thorwebdev
Copy link
Copy Markdown
Collaborator

@thorwebdev thorwebdev commented Mar 26, 2026

…or tool calling and response handling to support multiple function calls.
… proactive audio and affective dialog features, and refactor `turn_coverage` into `realtimeInputConfig`.
@thorwebdev thorwebdev merged commit 8bc190a into main Mar 26, 2026
2 checks passed
@thorwebdev thorwebdev deleted the thor/gemini-3-eap branch March 26, 2026 15:21
Copy link
Copy Markdown

@gemini-code-assist gemini-code-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.

Code Review

This pull request updates the default model to gemini-3.1-flash-live-preview across the repository and optimizes audio processing by reducing buffer sizes and eliminating memory allocations in the playback worklet. The WebSocket implementation was refactored to support bundled server responses and multiple tool calls, while proactivity and affective dialog features were removed. Feedback was provided to move an inline import in the Python SDK to the top of the file to adhere to PEP 8 standards.

await run_session()
except Exception as e:
logger.error(f"Error in Gemini session: {e}")
import traceback
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

While adding traceback for better error logging is a great improvement, it's best practice to place all imports at the top of the file. This follows PEP 8 guidelines (E402) and improves code readability and maintainability. Please move import traceback to the top of the module.

References
  1. PEP 8: E402 module level import not at top of file. Imports should always be at the top of the module, just after any module comments and docstrings, and before module globals and constants. (link)

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.

1 participant