Skip to content

Repeatkv transform#997

Open
quic-dhirajku wants to merge 5 commits into
quic:mainfrom
quic-dhirajku:repeatkv_transform
Open

Repeatkv transform#997
quic-dhirajku wants to merge 5 commits into
quic:mainfrom
quic-dhirajku:repeatkv_transform

Conversation

@quic-dhirajku
Copy link
Copy Markdown
Contributor

No description provided.

…VLMs. Based on PR quic#625. Addressed most of the comments made on the previous PR.

Repeat check is done on a subset of models during CI, primarily due to difference in configs of such models.

Signed-off-by: Dhiraj Kumar Sah <dhirajku@qti.qualcomm.com>
…ng with changes made for the new transforms.

TODO: Check for the ONNX directory path name being different.
Check if the list of classes for mapping covers all the models that we support.

Signed-off-by: Dhiraj Kumar Sah <dhirajku@qti.qualcomm.com>
…oder Wrappers were added to string mapping list to enable dummy model export for CI.

Changes were made to prevent multiple application of ReplicateKVTransform if done in either Encoder or Decoder Wrapper already.
Modeling files updated to access config in EncoderWrapper as well.
Infra added for causalLM and VLM checks for repeatKV setup CI tests.
CausalLM script APIRunner instantiation moved to allow updated input shapes to be made. Similarly commented export in VLM script since compile will call it with updated changes already.
TODO: Confirm the changes that were made for DeepSeekV3 model for RepeatKV, currently they were removed for a generic approach.

Signed-off-by: Dhiraj Kumar Sah <dhirajku@qti.qualcomm.com>
@quic-rishinr
Copy link
Copy Markdown
Contributor

@ochougul @vbaddi please review the PR

@quic-rishinr quic-rishinr added the 1.22 Release 1.22 candidate label May 25, 2026
Made changes to allow generic name based transformation of heads (num_attention_heads, n_heads, n_head etc).
Minor edits and utils created for this task.

Signed-off-by: Dhiraj Kumar Sah <dhirajku@qti.qualcomm.com>
Comment thread QEfficient/base/modeling_qeff.py
Edited the changes as suggested by quic-mamta.

Signed-off-by: Dhiraj Kumar Sah <dhirajku@qti.qualcomm.com>
@quic-dhirajku quic-dhirajku marked this pull request as ready for review May 27, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.22 Release 1.22 candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants