Skip to content

fix: [nvbugs/5287097] Align PP layer distribution between pytorch and TRT flow. #4399

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
May 19, 2025
Merged

fix: [nvbugs/5287097] Align PP layer distribution between pytorch and TRT flow. #4399

merged 4 commits into from
May 19, 2025

Conversation

yuxianq
Copy link
Collaborator

@yuxianq yuxianq commented May 16, 2025

No description provided.

@yuxianq yuxianq requested review from achartier, liji-nv and amukkara May 16, 2025 13:49
@yuxianq
Copy link
Collaborator Author

yuxianq commented May 16, 2025

/bot run --disable-fail-fast --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5515 [ run ] triggered by Bot

yuxianq
yuxianq commented May 16, 2025
View reviewed changes
achartier
achartier reviewed May 16, 2025
View reviewed changes
@amukkara amukkara mentioned this pull request May 16, 2025
Closed
amukkara
amukkara reviewed May 16, 2025
View reviewed changes
@tensorrt-cicd
Copy link
Collaborator

PR_Github #5515 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #4018 completed with status: 'FAILURE'

@yuxianq
Copy link
Collaborator Author

yuxianq commented May 17, 2025

/bot run --disable-fail-fast --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5551 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5551 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #4050 completed with status: 'FAILURE'

@yuxianq
Copy link
Collaborator Author

yuxianq commented May 17, 2025

/bot run --disable-fail-fast --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5559 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5559 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #4056 completed with status: 'FAILURE'

@chzblych
Copy link
Collaborator

/bot run --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5567 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5567 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #4062 completed with status: 'SUCCESS'

yuxianq added 2 commits May 19, 2025 02:13
@yuxianq
fix: Align PP layer distribution between pytorch and TRT flow.
4946fb6 
Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
@yuxianq
Address comments.
3574acc 
Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
@yuxianq yuxianq force-pushed the align-pp-layers branch from e29615e to 3574acc Compare May 19, 2025 02:15
@yuxianq
Copy link
Collaborator Author

yuxianq commented May 19, 2025

/bot run --add-multi-gpu-test

@yuxianq yuxianq requested review from achartier and amukkara May 19, 2025 02:23
@tensorrt-cicd
Copy link
Collaborator

PR_Github #5657 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5657 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #4131 completed with status: 'FAILURE'

@yuxianq
Fix CI error.
dccc457 
Signed-off-by: Yuxian Qiu <142763828+yuxianq@users.noreply.github.com>
@yuxianq
Copy link
Collaborator Author

yuxianq commented May 19, 2025

/bot run --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5671 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5671 [ run ] completed with state FAILURE
/LLM/main/L0_MergeRequest_PR pipeline #4142 completed with status: 'FAILURE'

@yuxianq
Copy link
Collaborator Author

yuxianq commented May 19, 2025

/bot run --add-multi-gpu-test

1 similar comment
@yuxianq
Copy link
Collaborator Author

yuxianq commented May 19, 2025

/bot run --add-multi-gpu-test

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5728 [ run ] triggered by Bot

@tensorrt-cicd
Copy link
Collaborator

PR_Github #5728 [ run ] completed with state SUCCESS
/LLM/main/L0_MergeRequest_PR pipeline #4186 completed with status: 'SUCCESS'

@achartier
Merge branch 'main' into align-pp-layers
64a92a1 
Signed-off-by: Aurelien Chartier <2567591+achartier@users.noreply.github.com>
@achartier
Copy link
Collaborator

/bot reuse-pipeline

@achartier achartier enabled auto-merge (squash) May 19, 2025 21:11
@tensorrt-cicd
Copy link
Collaborator

PR_Github #5775 [ reuse-pipeline ] triggered by Bot

@amukkara amukkara disabled auto-merge May 19, 2025 21:24
@tensorrt-cicd
Copy link
Collaborator

PR_Github #5775 [ reuse-pipeline ] completed with state SUCCESS
Reusing PR_Github #5728 for commit 64a92a1

@amukkara amukkara merged commit c8e062b into NVIDIA:main May 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@achartier achartier achartier approved these changes

@liji-nv liji-nv Awaiting requested review from liji-nv

@amukkara amukkara Awaiting requested review from amukkara

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@yuxianq @tensorrt-cicd @chzblych @achartier @amukkara