Skip to content

Modify the code to only keep Gemini 3 related setup and remove all Gemini 2.5 setup#6226

Open
shixiao-coder wants to merge 6 commits into
datacommonsorg:masterfrom
shixiao-coder:remove-gemini-3-feature-flag-code-citation
Open

Modify the code to only keep Gemini 3 related setup and remove all Gemini 2.5 setup#6226
shixiao-coder wants to merge 6 commits into
datacommonsorg:masterfrom
shixiao-coder:remove-gemini-3-feature-flag-code-citation

Conversation

@shixiao-coder
Copy link
Copy Markdown
Contributor

This PR update all calls that uses feature flag for Gemini 3 and removes all logics, constant related to Gemini 2.5

@shixiao-coder shixiao-coder requested a review from gmechali April 21, 2026 14:26
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

Gemini is experiencing higher than usual traffic and was unable to create the summary. Please try again in a few hours by commenting /gemini summary.

formatted_prompt=formatted_follow_up_questions_prompt,
schema=FollowUpQuestions,
gemini_model=gemini_model,
use_thinking_config=use_thinking_config)
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.

instead of setting use_think_config and gemini_model on lines 317 and 318, can you just set them here? Avoids redundancy

ENABLE_STAT_VAR_AUTOCOMPLETE = 'enable_stat_var_autocomplete'
ENABLE_NL_AGENT_DETECTOR = 'enable_nl_agent_detector'
NEW_RANKING_PAGE = 'new_ranking_page'
ENABLE_GEMINI_3_FLASH = 'enable_gemini_3_flash'
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.

After this PR is released to 2 prod cycles, make sure to delete the feature flag from the feature flag .json files!

2 prod cycles so that if we have to do a rollback at some point, it's still safe!

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