Skip to content

Merge Ulysses test fix to PR for full CI test#7785

Merged
tohtana merged 1 commit intodeepspeedai:tohtana/add_full_test_workflowfrom
tohtana:tohtana/fix-ulysses-peft-sp-group
Jan 17, 2026
Merged

Merge Ulysses test fix to PR for full CI test#7785
tohtana merged 1 commit intodeepspeedai:tohtana/add_full_test_workflowfrom
tohtana:tohtana/fix-ulysses-peft-sp-group

Conversation

@tohtana
Copy link
Collaborator

@tohtana tohtana commented Jan 17, 2026

Fix Ulysses test to unblock full test

Avoid relying on groups.mpu by validating the sequence parallel group
from the mpu returned by register_with_transformers.
@tohtana tohtana changed the title fix: use mpu for Ulysses PEFT SP group check Merge Ulysses test fix to PR for full CI test Jan 17, 2026
@tohtana tohtana merged commit df7eb63 into deepspeedai:tohtana/add_full_test_workflow Jan 17, 2026
17 checks passed
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.

1 participant