Skip to content

[SOT] Remove BreakGraph with paddle.maximum - #2731

Merged
Jiang-Jia-Jun merged 4 commits into
PaddlePaddle:developfrom
cattidea:sot_rm_if
Jul 8, 2025
Merged

[SOT] Remove BreakGraph with paddle.maximum#2731
Jiang-Jia-Jun merged 4 commits into
PaddlePaddle:developfrom
cattidea:sot_rm_if

Conversation

@DrRyanHuang

@DrRyanHuang DrRyanHuang commented Jul 7, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator 
        text_token_num = ((token_num - image_token_num) if
                          (token_num - image_token_num) > 0 else 1)

这部分的 if 语句导致的打断可以用 paddle.clip / paddle.maximum消除， token_num 和 image_token_num 都是整数

cc @SigureMo

@paddle-bot

paddle-bot Bot commented Jul 7, 2025

Copy link
Copy Markdown

Thanks for your contribution!

@DrRyanHuang DrRyanHuang mentioned this pull request Jul 7, 2025
Merged
SigureMo
SigureMo reviewed Jul 7, 2025
View reviewed changes
Comment thread fastdeploy/model_executor/models/ernie4_5_vl/ernie4_5_vl_moe.py Outdated
SigureMo
SigureMo approved these changes Jul 7, 2025
View reviewed changes

@SigureMo SigureMo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTMeow 🐾

标题记得更新一下

@DrRyanHuang DrRyanHuang changed the title [SOT] Remove BreakGraph with paddle.clip [SOT] Remove BreakGraph with paddle. maximum Jul 7, 2025
@DrRyanHuang DrRyanHuang changed the title [SOT] Remove BreakGraph with paddle. maximum [SOT] Remove BreakGraph with paddle. maximum Jul 7, 2025
@DrRyanHuang DrRyanHuang changed the title [SOT] Remove BreakGraph with paddle. maximum [SOT] Remove BreakGraph with paddle.maximum Jul 7, 2025
@Jiang-Jia-Jun
Jiang-Jia-Jun merged commit fefbd65 into PaddlePaddle:develop Jul 8, 2025
@DrRyanHuang
DrRyanHuang deleted the sot_rm_if branch July 8, 2025 04:41
xiaoguoguo626807 pushed a commit to xiaoguoguo626807/FastDeploy that referenced this pull request May 7, 2026
@DrRyanHuang
[SOT] Remove BreakGraph with paddle.maximum (PaddlePaddle#2731)
7f5b134 
* rm if with clip

* clip -> maximum

* int64 -> int32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SigureMo SigureMo SigureMo approved these changes
@xiaoxiaohehe001 xiaoxiaohehe001 xiaoxiaohehe001 approved these changes
@Jiang-Jia-Jun Jiang-Jia-Jun Jiang-Jia-Jun approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@DrRyanHuang @SigureMo @xiaoxiaohehe001 @Jiang-Jia-Jun