Skip to content

make vla server action_chunk and num_steps configurable#8

Draft
Sonorous281 wants to merge 1 commit into
RLinf:mainfrom
Sonorous281:feat/configurable-vla-action-chunk
Draft

make vla server action_chunk and num_steps configurable#8
Sonorous281 wants to merge 1 commit into
RLinf:mainfrom
Sonorous281:feat/configurable-vla-action-chunk

Conversation

@Sonorous281

Copy link
Copy Markdown
Contributor

Were hardcoded to 5/5; LIBERO-10 (long) runs at 10/4. Add --action-chunk and --num-steps, defaults 5/5 so existing launches are unchanged.

@qurakchin qurakchin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think it's better to add args in server function and add the config in client.

Were hardcoded to 5/5; LIBERO-10 (long) runs at 10/4. Add --action-chunk
and --num-steps, defaults 5/5 so existing launches are unchanged.
@Sonorous281
Sonorous281 force-pushed the feat/configurable-vla-action-chunk branch from 066f996 to 9a0ec0e Compare July 17, 2026 15:14
@Sonorous281

Copy link
Copy Markdown
Contributor Author

I think it's better to add args in server function and add the config in client.

It's just an optional flag. Doing it per-request from client would also need changes in the rlinf model code, since that's where action_chunk and num_steps actually get read from self.config, and that path is shared by all openpi evals. A run stays on one value anyway, so it doesn't really buy much.

@qurakchin

Copy link
Copy Markdown
Collaborator

It's not used now, and the flag is not good design. i trans it to a draft pr.

@qurakchin
qurakchin marked this pull request as draft July 19, 2026 09:32
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