Add Python package structure to models directory #9
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
models/andmodels/recursive_reasoning/directories lacked__init__.pyfiles, relying on implicit namespace packages. This PR adds proper package structure.Changes
models/__init__.py: Establishes models as a package, exports core utilities (EMA, layers, losses, sparse embedding)models/recursive_reasoning/__init__.py: Establishes recursive_reasoning subpackage, exports all model variants (TRM, HRM, transformers baseline, ERS_PMLL, etc.)Implementation Details
Both files define
__all__for API documentation but avoid eager imports to prevent torch dependency loading on package import:Backward compatible with existing dynamic model loading via
importlib.import_module()used in config-driven instantiation.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.