Skip to content

Return one train step output type from every trainer backend and scenario - #2150

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-147from
tom/refactor-miles/op13-148
Open

Return one train step output type from every trainer backend and scenario#2150
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-147from
tom/refactor-miles/op13-148

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Aug 4, 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-147 branch from 736ff0c to bd7ec5e Compare August 6, 2026 03:47
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from 448aab8 to 55df6d5 Compare August 6, 2026 03:47
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from bd7ec5e to baec89c Compare August 6, 2026 07:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch 2 times, most recently from 73294c6 to ef806bd Compare August 9, 2026 23:32
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 62b815a to 753dd27 Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from ef806bd to 322e378 Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 753dd27 to 99ed3a7 Compare August 10, 2026 06:01
@fzyzcjy
fzyzcjy requested a review from jybsuper as a code owner August 10, 2026 06:01
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch 3 times, most recently from f4a77df to a169744 Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 7e82262 to 67fafef Compare August 10, 2026 16:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from a169744 to 8bd91d2 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 67fafef to 8829029 Compare August 12, 2026 02:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from 8bd91d2 to c6bf462 Compare August 12, 2026 09:32
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 7abc2e2 to dda0e09 Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from c6bf462 to bf4454f Compare August 12, 2026 14:31
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from dda0e09 to c5c8577 Compare August 12, 2026 14:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch 2 times, most recently from 9f69242 to d6267ea Compare August 12, 2026 23:56
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 3a55156 to 826487b Compare August 13, 2026 01:13
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch 2 times, most recently from 2284250 to 9c23207 Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from 01fc3b1 to e1a1423 Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from 9c23207 to 3e2b7dd Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from e1a1423 to 8216e03 Compare August 13, 2026 06:00
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch 2 times, most recently from 05f086b to f90ac2d Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch 2 times, most recently from d29ccde to ade692c Compare August 14, 2026 04:46
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch 3 times, most recently from 40961bb to 7042fb6 Compare August 14, 2026 05:53
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from 7042fb6 to e06b8f7 Compare August 28, 2026 11:41
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-148 branch from e06b8f7 to cea62eb Compare August 29, 2026 12:38
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-147 branch from aa44a9b to 974ecff Compare August 29, 2026 12:38

@fzyzcjy fzyzcjy left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

No correctness or regression issue introduced by this commit remains at deliver-1. Subsequent return-value and external-data wiring is intentionally supplied by later operations and is outside this operation's scope.

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