Skip to content

JULES - Add build files#15

Draft
s2t2 wants to merge 2 commits intocopybara_pushfrom
add-build-files
Draft

JULES - Add build files#15
s2t2 wants to merge 2 commits intocopybara_pushfrom
add-build-files

Conversation

@s2t2
Copy link
Owner

@s2t2 s2t2 commented Jun 2, 2025

No description provided.

…ing` directory to include the Python files in that module and its subdirectories in the Bazel build.

The new BUILD file defines `pytype_strict_library` targets for all Python files and includes dependencies for each target.

No other BUILD files were modified as they already covered all other Python files in the project.
…module.

This change implements your feedback to have separate BUILD files in each subdirectory of the `smart_control/reinforcement_learning` module.

Here's what I did:
- I removed the monolithic BUILD file from `smart_control/reinforcement_learning`.
- I added new BUILD files to each of the following subdirectories:
    - `smart_control/reinforcement_learning/agents/networks/`
    - `smart_control/reinforcement_learning/agents/`
    - `smart_control/reinforcement_learning/observers/`
    - `smart_control/reinforcement_learning/policies/`
    - `smart_control/reinforcement_learning/replay_buffer/`
    - `smart_control/reinforcement_learning/scripts/`
    - `smart_control/reinforcement_learning/utils/`
- Each new BUILD file defines `pytype_strict_library` targets for the Python files in its respective directory and includes dependencies, including those between these subdirectories.
- I reviewed other BUILD files in the project; no further updates were necessary.
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