Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HITL - Move BaselinesController to its own file. #2122

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

0mdc
Copy link
Contributor

@0mdc 0mdc commented Dec 16, 2024

Motivation and Context

This changeset moves BaselinesController to its own file.
Currently, this is a noop. However, it will allow for creating lighter containers by decoupling habitat_baselines from HITL.

How Has This Been Tested

Tested locally.

Types of changes

  • [Refactoring]

Checklist

  • My code follows the code style of this project.
  • I have updated the documentation if required.
  • I have read the CONTRIBUTING document.
  • I have completed my CLA (see CONTRIBUTING)
  • I have added tests to cover my changes if required.

@0mdc 0mdc requested a review from aclegg3 December 16, 2024 23:57
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Dec 16, 2024
@0mdc 0mdc force-pushed the 0mdc/move_baselines_controller branch from 5b5d701 to ddbd084 Compare January 20, 2025 22:45
@0mdc 0mdc merged commit 2ad554b into main Jan 21, 2025
4 checks passed
@0mdc 0mdc deleted the 0mdc/move_baselines_controller branch January 21, 2025 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants