add Step-3.5-Flash model support#18084
Conversation
Co-authored-by: ltd0924 <[email protected]>
Summary of ChangesHello @yhyang201, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces full support for the Step3.5 Flash model, encompassing its base architecture and a specialized Multi-Token Predictor (MTP) variant. It includes new configuration definitions, detailed implementations of attention and Mixture-of-Experts (MoE) layers, and integrates the model with speculative decoding capabilities. The changes ensure the new model can be properly configured, loaded, and utilized within the system, with a noted temporary design choice in MTP hidden state handling that will be addressed in future updates. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
/tag-and-rerun-ci |
There was a problem hiding this comment.
Code Review
This pull request introduces support for the Step3.5 flash model and its MTP variant, which is a significant feature addition. The changes are comprehensive, touching upon model configurations, implementations, and parsers. While the overall implementation appears solid, I've identified a few areas for improvement, including a potential runtime error due to a type mismatch in the model configuration, a possible TypeError from an unsafe dictionary access, and some opportunities to improve code clarity by removing redundancy and hardcoded values.
|
/rerun-failed-ci |
Co-authored-by: ltd0924 <[email protected]>
Co-authored-by: ltd0924 <[email protected]>
Motivation
add Step-3.5-Flash model support
Important notes:
The current implementation differs slightly from the standard MTP implementation in Step3.5 Flash. In the standard multi-layer MTP design of Step3.5 Flash, the hidden states of each MTP layer are passed from the preceding MTP layer (the hidden states of the initial (layer-0) MTP still being provided by the target model). In contrast, the current SGL implementation obtains hidden states directly from the target model for all MTP layers. Empirical evaluations indicate that the overall performance remains strong; however, this design choice may lead to a slight reduction in acceptance rate in certain scenarios. This behavior will be corrected shortly, and we expect to implement the standard multi-layer MTP design of Step3.5 Flash in the near future.
Modifications
Accuracy Tests
Benchmarking and Profiling
Checklist
Review Process
/tag-run-ci-label,/rerun-failed-ci,/tag-and-rerun-ci