Skip to content

[misc] fix: add missing __init__.py files to package directories#5209

Open
guillemgt wants to merge 2 commits intoverl-project:mainfrom
guillemgt:add-missing-init-files
Open

[misc] fix: add missing __init__.py files to package directories#5209
guillemgt wants to merge 2 commits intoverl-project:mainfrom
guillemgt:add-missing-init-files

Conversation

@guillemgt
Copy link

What does this PR do?

Adds missing __init__.py files to 11 package directories that contained Python modules but were not properly configured as packages. This ensures these directories are recognized as importable Python packages.

Affected directories:

  • Several directories in experimental/ (fully_async_policy, one_step_off_policy, vla/*, reward_loop/router, transfer_queue)
  • tools/utils/mcp_clients/
  • utils/sglang/
  • workers/rollout/trtllm_rollout/

All files include proper Apache 2.0 license headers consistent with the project.

Checklist Before Starting

Test

Confirmed that affected modules can be imported after the change.
No functional changes - only adds empty files with license headers. Existing tests should continue to pass.

Checklist Before Submitting

  • Read the Contribute Guide
  • Applied pre-commit checks (all relevant checks passed)
  • Documentation not needed (no user-facing changes)
  • No tests needed (structural fix only, no functional changes)

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly adds missing __init__.py files to several directories, ensuring they are properly recognized as Python packages. This is a good structural improvement that enhances the project's packaging and import behavior. The changes are straightforward and correct, and I have no further comments.

@@ -0,0 +1,14 @@
# Copyright 2025 Bytedance Ltd. and/or its affiliates
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is contributed by Nvidia

Copy link
Author

Choose a reason for hiding this comment

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

I see. I've updated the license to match the other files in the directory, however they also don't list Nvidia

@@ -0,0 +1,14 @@
# Copyright 2025 Bytedance Ltd. and/or its affiliates
# Copyright 2025 Meituan Ltd. and/or its affiliates
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is VLA contributed by Meituan?

Copy link
Author

Choose a reason for hiding this comment

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

No, sorry, fixed this now

@CLAassistant
Copy link

CLAassistant commented Feb 6, 2026

CLA assistant check
All committers have signed the CLA.

@guillemgt guillemgt force-pushed the add-missing-init-files branch from 20d6eb1 to 69df06f Compare February 6, 2026 11:52
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.

3 participants