Skip to content

.NET: use valid Gemini model - #2695

Merged
Roger Barreto (rogerbarreto) merged 4 commits into
microsoft:mainfrom
jochenkirstaetter:fix-gemini-model
Dec 10, 2025
Merged

.NET: use valid Gemini model#2695
Roger Barreto (rogerbarreto) merged 4 commits into
microsoft:mainfrom
jochenkirstaetter:fix-gemini-model

Conversation

@jochenkirstaetter

@jochenkirstaetterJochen Kirstätter (jochenkirstaetter) commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

Motivation and Context

I saw the initial PR to add Gemini Agent providers to the repository and tested it given the PR source code. However, the referenced Gemini model gemini-2.5-fast does not exist any more. I updated it to gemini-2.5-flash instead.

Description

Change the name of the Gemini model to an existing Flash 2.5 model. The previous reference to fast is not valid any more.

Contribution Checklist

  • [ X ] The code builds clean without any errors or warnings
  • [ X ] The PR follows the Contribution Guidelines
  • [ X ] All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

CopilotAI review requested due to automatic review settings December 8, 2025 11:58
@markwallace-microsoftMark Wallace (markwallace-microsoft) added the .NET Usage: [Issues, PRs], Target: .Net label Dec 8, 2025
@github-actionsgithub-actionsBot changed the title use proper Gemini model.NET: use proper Gemini modelDec 8, 2025
@jochenkirstaetterJochen Kirstätter (jochenkirstaetter) changed the title .NET: use proper Gemini model.NET: use valid Gemini modelDec 8, 2025
@jochenkirstaetter

Jochen Kirstätter (jochenkirstaetter) commented Dec 8, 2025

Copy link
Copy Markdown
ContributorAuthor

Note: I'm the maintainer of "a community driven Mscc.GenerativeAI.Microsoft package"

Cheers

CopilotAI reviewed Dec 8, 2025
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects the default Google Gemini model name from the non-existent gemini-2.5-fast to the valid gemini-2.5-flash model in the sample code.

  • Updates the default model name to a valid Gemini 2.5 Flash model

@jochenkirstaetter

Jochen Kirstätter (jochenkirstaetter) commented Dec 8, 2025•
edited
Loading

Copy link
Copy Markdown
ContributorAuthor

Use named parameters explicitly to avoid ambiguous constructor methods because my package supports Google AI and Vertex AI interfaces. Also, ILogger interface is supported.

Cheers

github-merge-queueBot pushed a commit that referenced this pull request Dec 10, 2025
@jochenkirstaetter@rogerbarreto
.NET: use valid Gemini model (#2695)
e16353f
* use proper Gemini model
* use named parameters
---------
Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com>
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks Dec 10, 2025
@github-merge-queue
github-merge-queueBot removed this pull request from the merge queue due to failed status checks Dec 10, 2025
Merged via the queue into microsoft:main with commit 0d9ae19Dec 10, 2025
14 checks passed
This was referenced Dec 21, 2025
Merged
Merged
Closed
Closed
Closed
Closed
This was referenced Jan 9, 2026
Closed
Closed
Closed
This was referenced Jan 19, 2026
Closed
Closed
Closed
Closed
Merged
Closed
Closed
Merged
Closed
Closed
Closed
This was referenced Jan 28, 2026
Closed
Closed
Closed
Open
Closed
Aris Nguyen (arisng) pushed a commit to arisng/agent-framework that referenced this pull request Feb 2, 2026
@jochenkirstaetter@rogerbarreto
.NET: use valid Gemini model (microsoft#2695)
8b885c1
* use proper Gemini model
* use named parameters
---------
Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com>
This was referenced Feb 2, 2026
Open
Closed
@dependabotdependabotBot mentioned this pull request Feb 16, 2026
Closed
This was referenced Feb 23, 2026
Closed
Open
This was referenced Mar 2, 2026
Closed
Open
Open
@dependabotdependabotBot mentioned this pull request Mar 16, 2026
Merged
@dependabotdependabotBot mentioned this pull request May 2, 2026
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@rogerbarretoRoger Barreto (rogerbarreto)rogerbarreto approved these changes
+1 more reviewer
@markwallace-microsoftMark Wallace (markwallace-microsoft)markwallace-microsoft approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

.NETUsage: [Issues, PRs], Target: .Net

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jochenkirstaetter@rogerbarreto@markwallace-microsoft