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

thrusterPlatformReference refactor #489

Closed
rcalaon opened this issue Nov 15, 2023 · 0 comments · Fixed by #490
Closed

thrusterPlatformReference refactor #489

rcalaon opened this issue Nov 15, 2023 · 0 comments · Fixed by #490
Assignees
Labels
enhancement New feature or request refactor Clean up with no new functionality

Comments

@rcalaon
Copy link
Contributor

rcalaon commented Nov 15, 2023

Describe your use case
Refactoring the thrusterPlatformReference module in order to read in a message containing the static information on thrust vector and application point in platform frame coordinates, and outputting the same information in body frame coordinates and with respect to the origin of the B frame.

Moreover, this issue aims to add an integral feedback to the control law in thrusterPlatformReference that uses the thruster platform to dump momentum on the reaction wheels. Additionally, a user input is added to provide a limit to the gimbal angles allowed.

Describe alternatives solutions you've considered
An alternative solution consists in using only the proportional feedback term combined with continuous center of mass (CoM) estimation.

Additional context
When the CoM is not continuously estimated, the bias on the CoM knowledge causes a steady-state bias on the wheel speeds, which cannot be dumped by the control law. Adding the integral feedback term aims to dump that steady-state bias.

@rcalaon rcalaon added enhancement New feature or request refactor Clean up with no new functionality labels Nov 15, 2023
@rcalaon rcalaon self-assigned this Nov 15, 2023
@rcalaon rcalaon added this to Basilisk Nov 15, 2023
@rcalaon rcalaon linked a pull request Nov 16, 2023 that will close this issue
@rcalaon rcalaon moved this to 👀 In review in Basilisk Nov 16, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Basilisk Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor Clean up with no new functionality
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant