Skip to content

Update to gemini-3.5-flash-lite - #1870

Merged
paulb777 merged 1 commit into
mainfrom
pb-model-updates
Sep 8, 2026
Merged

paulb777 merged 1 commit into
mainfrom
pb-model-updates

Conversation

@paulb777

@paulb777 paulb777 commented Sep 8, 2026

Copy link
Copy Markdown
Member

Update to gemini-3.5-flash-lite and gemini-3.1-flash-image

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

Code Review

This pull request updates the default Gemini model from "gemini-3.1-flash-lite" to "gemini-3.5-flash-lite" across multiple view models and sample configurations, and updates the image generation sample to use "gemini-3.1-flash-image". The reviewer suggested defining a single static constant for the default model name to avoid duplication and improve maintainability.

useCases: [UseCase],
navRoute: String,
modelName: String = "gemini-3.1-flash-lite",
modelName: String = "gemini-3.5-flash-lite",

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.

medium

The default model name "gemini-3.5-flash-lite" is hardcoded here and duplicated across four other files (ChatViewModel.swift, FunctionCallingViewModel.swift, GroundingViewModel.swift, and MultimodalViewModel.swift). To improve maintainability and ensure a single source of truth, consider defining a static constant on Sample (e.g., public static let defaultModelName = "gemini-3.5-flash-lite") and referencing it across all these files. This will make future model updates much easier and less error-prone.

@paulb777
paulb777 merged commit a4c3a7f into main Sep 8, 2026
18 checks passed
@paulb777
paulb777 deleted the pb-model-updates branch September 8, 2026 18:00
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