+
Skip to content

Conversation

bsowell
Copy link
Contributor

@bsowell bsowell commented Jun 17, 2025

Drops preview label from Gemini 2.5 Flash and Gemini 2.5 Pro, and adds support for Gemini 2.5 Flash Lite Preview.

Drops preview label from Gemini 2.5 Flash and Gemini 2.5 Pro, and adds support
for Gemini 2.5 Flash Lite Preview.
@bsowell bsowell requested review from Copilot and karanataryn June 17, 2025 19:31
Copy link

@Copilot Copilot AI left a comment

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 updates the GeminiModels enum to remove explicit “preview” versions for Flash and Pro, alias those preview constants to the stable models, and adds support for a new Flash Lite preview.

  • Replace -preview-<date> constants for Gemini 2.5 Flash and Pro with stable model constants and alias old preview names.
  • Introduce GEMINI_2_5_FLASH_LITE_PREVIEW for the new Gemini 2.5 Flash Lite Preview.
Comments suppressed due to low confidence (2)

lib/sycamore/sycamore/llms/config.py:66

  • The new GEMINI_2_5_FLASH_LITE_PREVIEW model should be added to your unit tests to verify it resolves correctly and prevent future regressions.
    GEMINI_2_5_FLASH_LITE_PREVIEW = GeminiModel(name="gemini-2.5-flash-lite-preview-06-17", is_chat=True)

lib/sycamore/sycamore/llms/config.py:60

  • [nitpick] Since the version date suffix has been dropped for Flash, consider updating the enum class docstring or adding an inline comment to explain that the unversioned model name now points to the latest release.
    GEMINI_2_5_FLASH = GeminiModel(name="gemini-2.5-flash", is_chat=True)

GEMINI_2_5_FLASH_PREVIEW = GeminiModel(name="gemini-2.5-flash-preview-05-20", is_chat=True)
GEMINI_2_5_PRO_PREVIEW = GeminiModel(name="gemini-2.5-pro-preview-05-06", is_chat=True)
GEMINI_2_5_FLASH = GeminiModel(name="gemini-2.5-flash", is_chat=True)
GEMINI_2_5_FLASH_PREVIEW = GEMINI_2_5_FLASH # Alias for the preview model
Copy link

Copilot AI Jun 17, 2025

Choose a reason for hiding this comment

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

[nitpick] Aliasing the preview constant to the stable Flash model may be confusing. Consider either removing the alias and using the stable constant directly or enhancing the enum’s docstring to explain that preview now defaults to the stable endpoint.

Copilot uses AI. Check for mistakes.

GEMINI_2_5_FLASH_PREVIEW = GEMINI_2_5_FLASH # Alias for the preview model

GEMINI_2_5_PRO = GeminiModel(name="gemini-2.5-pro", is_chat=True)
GEMINI_2_5_PRO_PREVIEW = GEMINI_2_5_PRO # Alias for the preview model
Copy link

Copilot AI Jun 17, 2025

Choose a reason for hiding this comment

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

[nitpick] Similar to Flash, the Pro preview is aliased to the stable Pro model. This might obscure intent; update documentation or remove the alias to maintain clarity.

Suggested change
GEMINI_2_5_PRO_PREVIEW = GEMINI_2_5_PRO # Alias for the preview model
GEMINI_2_5_PRO_PREVIEW = GeminiModel(name="gemini-2.5-pro-preview", is_chat=True) # Explicitly define the preview model

Copilot uses AI. Check for mistakes.

@bsowell bsowell merged commit 060f6bb into main Jun 17, 2025
16 of 31 checks passed
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

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载