Skip to content

Conversation

@wangyb-A
Copy link
Contributor

@wangyb-A wangyb-A commented Sep 23, 2025

Adding replay mode for child context

  • For checkpoint after func call succeeded, if the result exceeded the checkpoint size limit, we will set replay mode to True, and the checkpoint payload will be either be "" or summarized result (if summary generator is provided).

  • When replay, if the replay mode is True, we will re-execute the child func without checkpointing.

Hatch tests are passing locally!

@wangyb-A wangyb-A requested a review from yaythomas September 23, 2025 20:07
@wangyb-A wangyb-A marked this pull request as ready for review September 23, 2025 20:15
Copy link
Member

@yaythomas yaythomas left a comment

Choose a reason for hiding this comment

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

conventional commit msgs please. https://www.conventionalcommits.org/en/v1.0.0/

- During child execution, if the result exceeded checkpoint api size limit, set replay mode to True
- When replay, if replay mode is true, re-execute func instead of return the result from the payload
@wangyb-A wangyb-A requested a review from yaythomas September 24, 2025 00:11
Copy link
Member

@yaythomas yaythomas left a comment

Choose a reason for hiding this comment

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

:shipit:

remember to squash commits pls and retain conventional commit msg

@wangyb-A wangyb-A merged commit 850bd07 into main Sep 24, 2025
3 of 7 checks passed
@wangyb-A wangyb-A deleted the feat/replay_children branch September 24, 2025 00:29
This was referenced Sep 24, 2025
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