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

updated drytest to select whatever thrusters you want continuously #611

Merged
merged 6 commits into from
Aug 22, 2024

Conversation

NathanielFactor
Copy link
Contributor

Related Issue(s)

Other Dependent PRs

Changes Made

  • Describe the changes made in this PR.

Rationale/Other Solutions Considered

  • Explain the rationale behind this PR.
  • Mention any alternative solutions considered (if any).

Tests Done

  • Describe the tests that have been performed.
  • Mention any relevant test cases or scenarios.

Optional Screen Recording

Attach a screen recording of test run(s) (if applicable).

Required Wiki Changes (if any)

  • List any changes that need to be made to the project's wiki (if applicable).

Required install_dependencies.sh Changes (if any)

  • List any new dependencies that need to be installed by the install_dependencies.sh script

Related Branches (other than one that will be merged)

  • List any related branches (if applicable).

@NathanielFactor
Copy link
Contributor Author

fixed CI w new repo name, also committed new changes to dry test. Should be good now I think, can't test in sim until later tonight but I think its fine.

@Rain1618 Rain1618 merged commit 0e94c31 into noetic Aug 22, 2024
2 checks passed
@Rain1618 Rain1618 deleted the optimized_dry_test branch August 22, 2024 19:07
Alb33rt pushed a commit that referenced this pull request Sep 21, 2024
)

* updated drytest to select whatever thrusters you want continuously

* updated optimized dry test

* updated CI

* test 2

* removed fowards test

* updated docker volumes

---------

Co-authored-by: Nathaniel Factor <NRF@kkbl.com>
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