Skip to content

Grounding metadata#2

Open
getchannel wants to merge 37 commits intomainfrom
groundingMetadata
Open

Grounding metadata#2
getchannel wants to merge 37 commits intomainfrom
groundingMetadata

Conversation

@getchannel
Copy link
Owner

Please describe the changes in your PR. If it is addressing an issue, please reference that as well.

getchannel and others added 30 commits May 9, 2025 10:50
added proper .py to file name.
This is an example to test usage of the Files API integration. Specifically with the Gemini Multimodal Live Service.
unecessary import removed.
- Remove debug logging from grounding metadata event handlers
- Simplify logging in _process_grounding_metadata method
- Clean up example file logging for better readability
- Remove verbose event parsing comments

Based on suggestions from draft PR pipecat-ai#2121
- Reorganize imports and transport_params structure
- Remove copyright header for consistency
- Enhance grounding metadata logging with better formatting
- Remove unnecessary PipelineParams configuration
- Update message content formatting

Completes incorporation of draft PR pipecat-ai#2121 changes
- Change function body to use 'str' parameter consistently
- Matches pattern used in OpenAI Realtime Beta service
- Fixes bug where parameter was named 'str' but body used 'message_str'
- Maintains consistency with existing codebase patterns
tab indentation fix
-FunctionCallFromLLM
-Delete duplicate Gemini imports
getchannel and others added 7 commits July 20, 2025 18:27
- Clean up unnecessary whitespace in `gemini.py`, `events.py`, and `file_api.py`
- Ensure consistent formatting in `26g-gemini-multimodal-live-groundingMetadata.py`
- Improve readability by aligning code and removing trailing spaces
- Remove duplicate import in __init__.py
- Clean up extra blank lines in gemini.py
- Remove extra blank line in _create_single_response method
- Remove duplicate file_api initialization lines
- Keep grounding metadata tracking functionality
- Maintain clean code structure
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