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.

Sign up for GitHub

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

Jump to bottom

Use ECS to pass thruster commands #2434

Open
arjo129 opened this issue Jun 6, 2024 · 0 comments
Open

Use ECS to pass thruster commands #2434

arjo129 opened this issue Jun 6, 2024 · 0 comments
Labels
enhancement New feature or request help wanted We accept pull requests!

Comments

@arjo129
Copy link
Contributor

arjo129 commented Jun 6, 2024

Desired behavior

Currently systems like the thruster plugin use topics to achieve desired speeds. This means that when we test them we have to rely on a wide variety of synchronization tricks to make sure that the tests pass. This adds additional load on our CI. It also mean integrating with other types of autopilot software is very painful.

Implementation suggestion

Add support for passing thruster commands via the ECS.
@arjo129 arjo129 added the enhancement New feature or request label Jun 6, 2024
@arjo129 arjo129 changed the title More deterministic thruster behaviour Use ECS to pass thruster commands Jun 6, 2024
@azeey azeey moved this from Inbox to To do in Core development Jul 22, 2024
@azeey azeey added help wanted We accept pull requests! and removed help wanted We accept pull requests! labels Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request help wanted We accept pull requests!
Projects
Core development
Status: To do
Milestone
No milestone
Development

No branches or pull requests
2 participants
@azeey @arjo129