Skip to content

Conversation

@Darktex
Copy link
Contributor

@Darktex Darktex commented Oct 3, 2025

No description provided.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 3, 2025
Copy link
Contributor

Choose a reason for hiding this comment

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

Did we need to check in cpython files?



@dataclass
class ExecutionResult:
Copy link
Contributor

Choose a reason for hiding this comment

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

A few points to consider:

  1. stdout and stderr could be long streams and may cause env container to OOM if we store it in memory. Let's discuss on how the policy would leverage this information. Better to minimize the context sharing from inside and outside of the container.
  2. One of the paradigms we are seeing with SWE agent training is that exit_code, failure reason are generally a good starting point for execution result. Lets discuss whether this paradigm can be applied here too.

@Darktex Darktex closed this Oct 6, 2025
@Darktex Darktex reopened this Oct 6, 2025
@Darktex Darktex merged commit 1b6e3ff into main Oct 6, 2025
1 check passed
jspisak pushed a commit that referenced this pull request Oct 22, 2025
Updating list of supporters with LastMile AI
pankit-eng pushed a commit that referenced this pull request Nov 3, 2025
FIX: Handle double-nested observation in client parser
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants