Skip to content

[API] Support background experiment execution - #1062

Closed
podkidyshev wants to merge 1 commit into
ipod/api-text-inputsfrom
ipod/api-background
Closed

podkidyshev wants to merge 1 commit into
ipod/api-text-inputsfrom
ipod/api-background

Conversation

@podkidyshev

Copy link
Copy Markdown
Contributor

Summary

  • Add wait=False to return a pending experiment and execute it in a separate worker process.
  • Persist the worker request and controller log in the experiment directory. Record setup and execution failures in experiment output.
  • Extend existing API tests and documentation for background execution. Synchronous execution remains the default.

Test Plan

  • Automated CI.
  • Manual macOS test: launched two one-second standalone Sleep experiments from a short-lived Python process, using Path inputs for one and TOML text for the other. Both returned pending snapshots. After the caller exited, a separate process verified both completed, each with one successful run, timestamps, controller log, and saved request.
PASS: both background workers completed after caller exit; one successful run and timestamps each

Additional Notes

Based on #1061. This is the final split of the original API implementation. Workers load installed CloudAI plugins in a fresh process; caller-only in-memory registrations are not transferred.

@coderabbitai

coderabbitai Bot commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: Ivan Podkidyshev <ipodkidyshev@nvidia.com>
@podkidyshev podkidyshev closed this Oct 9, 2026
@podkidyshev
podkidyshev deleted the ipod/api-background branch October 9, 2026 11:03
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