Skip to content

Feature/detached update 0414#70

Open
scflow wants to merge 32 commits intoNVlabs:mainfrom
scflow:feature/detached-update-0414
Open

Feature/detached update 0414#70
scflow wants to merge 32 commits intoNVlabs:mainfrom
scflow:feature/detached-update-0414

Conversation

@scflow
Copy link
Copy Markdown

@scflow scflow commented Apr 14, 2026

No description provided.

Add .github/workflows/lint.yml (pre-commit on push to main and on PRs)
Add scripts/setup-github-runner.sh for minimal runner install
Add docs for runner setup and fork PR approval
Add GITHUB_CI_MIGRATION_PLAN for future CI parity with GitLab
@mwatson-nvidia
Copy link
Copy Markdown
Collaborator

It took a while to understand the PR, and, after understanding it at a high level, I'm not sure it's a good fit for the repo.

From what I can tell, this adds

  1. a classical planner -- this seems a bit out of place in the repo which is really targeted at end-to-end model simulation
  2. interactivity in the runtime - this use case seems like the closest fit, and there are some internal developments that might address things like live viewing (AlpaDreams), although probably with a different approach (minimal changes to the existing API)
  3. checkpointing - I think there are some issues with the architecture that make this a tough case (state management across grpc boundaries)
  4. decision orchestration - an interesting feature, but might be better solved behind the existing runtime-driver API

If there are pieces of this you'd like to contribute separately, we'd be happy to look at smaller, focused PRs

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.

4 participants