Skip to content

GML-2164 Fix incorrect OpenAI label in GenAI and Groq instantiation logs#50

Open
prinskumar-tigergraph wants to merge 1 commit into
release_2.0.1from
GML-2164-Model-logs-correction
Open

GML-2164 Fix incorrect OpenAI label in GenAI and Groq instantiation logs#50
prinskumar-tigergraph wants to merge 1 commit into
release_2.0.1from
GML-2164-Model-logs-correction

Conversation

@prinskumar-tigergraph

@prinskumar-tigergraph prinskumar-tigergraph commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

PR Type

Bug fix


Description

  • Correct Google GenAI instantiation logging

  • Correct Groq instantiation logging


Diagram Walkthrough

flowchart LR
  A["LLM service initialization"] -- "logs provider" --> B["Correct provider-specific label"]
  B -- "Google service" --> C["GoogleGenAI"]
  B -- "Groq service" --> D["Groq"]
Loading

File Walkthrough

Relevant files
Bug fix
google_genai_service.py
Correct Google GenAI log provider label                                   

common/llm_services/google_genai_service.py

  • Updates the initialization log message.
  • Replaces incorrect OpenAI provider label with GoogleGenAI.
+1/-1     
groq_llm_service.py
Correct Groq log provider label                                                   

common/llm_services/groq_llm_service.py

  • Updates the initialization log message.
  • Replaces incorrect OpenAI provider label with Groq.
+1/-1     

@tg-pr-agent

tg-pr-agent Bot commented Jul 13, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@tg-pr-agent

tg-pr-agent Bot commented Jul 13, 2026

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

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