Skip to content

Pass node_rank instead of rank when computing the engine launch command - #2003

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-51from
tom/refactor-miles/op13-48
Open

Pass node_rank instead of rank when computing the engine launch command#2003
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-51from
tom/refactor-miles/op13-48

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Part of #1837

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 394e00b to 6fd1deb Compare August 2, 2026 12:40
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 3 times, most recently from 9f114ae to b45bd94 Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 37bcb24 to a0f6fb2 Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from b45bd94 to 49585ab Compare August 4, 2026 06:54
@fzyzcjy
fzyzcjy requested a review from Rockdu as a code owner August 4, 2026 06:54
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from a0f6fb2 to e112a45 Compare August 4, 2026 06:54
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 49585ab to a2ae9e9 Compare August 4, 2026 07:26
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from e112a45 to 4915e35 Compare August 4, 2026 07:26
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from a2ae9e9 to a45e897 Compare August 4, 2026 07:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 4915e35 to 9a7e1a4 Compare August 4, 2026 07:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from a45e897 to 939d1ba Compare August 6, 2026 03:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch 2 times, most recently from 92b8cfb to 9f8b0ba Compare August 6, 2026 07:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 939d1ba to 9778253 Compare August 6, 2026 07:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 9f8b0ba to 68cdef2 Compare August 9, 2026 23:32
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 2 times, most recently from 442059f to ae7dfac Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 68cdef2 to a39af60 Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from ae7dfac to 80865e8 Compare August 10, 2026 06:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from a39af60 to 6ec881c Compare August 10, 2026 06:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 80865e8 to 8c60a36 Compare August 10, 2026 16:06
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 18866e8 to 5cc15cc Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 8c60a36 to 542579e Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 5cc15cc to faffae1 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 3 times, most recently from 75a9917 to ce2e105 Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 41f958d to 4ec664c Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from ce2e105 to 23be9b2 Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 4ec664c to 2969917 Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 23be9b2 to 02cd54f Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 2969917 to 6e19c4c Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 02cd54f to 04d842a Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 6e19c4c to a978a5d Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 04d842a to 9f4cf59 Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from a978a5d to 40c3afb Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 3 times, most recently from d0ec1f2 to b711af0 Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 121a3cd to 7d4b538 Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from b711af0 to b4d8f53 Compare August 14, 2026 04:46

@guapisolo guapisolo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Stack-verified: #2063 removes the obsolete global-rank alignment guard, and #2096 repairs the stale test caller and adds cross-cell coverage. Both fixes land before #2176 and remain intact through #2649.

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.

2 participants