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

Thruster and Fin Class #16

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

BradenMeyers
Copy link
Contributor

I prefer this method over the other pull request I made that just does the fin class. If you perfer to leave the propeller dynamics inside the vehicle dynamics than just use the changes in the other pull request. I tried my best to follow conventions, comments, and variable names.

The one thing I am unsure about is how to handle the convention for fin rotation. The way I did it was a CCW rotation (looking from the top of the fin) as positive delta.

Copy of the REMUS 100 dynamics with modification for the forces applied by the fins

  • New thruster class that takes care of all propeller dynamics outside in a separate file
  • New fin class that can be placed at a specific location on the vehicle.
  • Calculates force vector given the relative velocity
  • Method to handle actuator dynamics given commands to the fin

Verified that results from autopilot gives the same results as the REMUS 100.

Let me know what you think and if there is a different way I should approach this.

Thanks,
Braden Meyers

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