Skip to content

Add assertions to reject training data generated under the sglang default weight version - #1894

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op1-8from
tom/refactor-miles/op1-14
Open

Add assertions to reject training data generated under the sglang default weight version#1894
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op1-8from
tom/refactor-miles/op1-14

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Jul 28, 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/op1-8 branch from e4de483 to b7ac6fc Compare July 29, 2026 04:10
@fzyzcjy
fzyzcjy requested a review from yushengsu-thu as a code owner July 29, 2026 04:10
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch 2 times, most recently from aea9173 to 5944546 Compare August 3, 2026 23:43
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from 09fcdad to 8915cff Compare August 4, 2026 00:38
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch 2 times, most recently from 7487e46 to 267cd10 Compare August 4, 2026 06:52
@fzyzcjy
fzyzcjy requested a review from Rockdu as a code owner August 4, 2026 06:52
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch 2 times, most recently from 4834d96 to 6665c38 Compare August 6, 2026 07:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch from 267cd10 to 72943b4 Compare August 6, 2026 07:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from 6665c38 to e3047fa Compare August 9, 2026 23:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch 2 times, most recently from b751307 to fe83e47 Compare August 10, 2026 05:59
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from e3047fa to dfd4a0b Compare August 10, 2026 05:59
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch from fe83e47 to 21f9374 Compare August 10, 2026 16:06
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch 2 times, most recently from 0dcbfaf to f74ffb6 Compare August 10, 2026 16:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch 3 times, most recently from 80d85c5 to f5965ec Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from 198de11 to daba386 Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from daba386 to f21da15 Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch 2 times, most recently from d1f6d17 to d3ed573 Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from 8be9654 to e2b0e70 Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch from d3ed573 to 2c79fef Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-8 branch from e2b0e70 to 1f9b0c7 Compare August 13, 2026 06:00
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch from 2c79fef to 11415e1 Compare August 13, 2026 06:00
…ault weight version

Every span on incoming training samples must differ from the sglang
"default" placeholder and match the numeric pattern miles stamps on
weight updates; debug modes and LoRA are exempt. Covered by pure
function tests and a manager-level regression.
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op1-14 branch from 11415e1 to e50957c Compare August 14, 2026 04:45

@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.

Reviewed in the #1837 stack through #2649, including the #2176 safe-merge boundary. The guard rejects uninitialized SGLang weight versions without rejecting fully-async numeric staleness; no unresolved P1 remains, with missing-provenance and CI-injection coverage tracked as P2 follow-ups.

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