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

Refactor BoatState class for integration #312

Merged
merged 5 commits into from
Jun 10, 2024
Merged

Conversation

DFriend01
Copy link
Contributor

Description

  • Made the compute_net_force_and_torque function private and called it in the step function.
  • Modified the step function to accept true wind and current velocities in the global reference frame.
  • Modified the compute_net_force_and_torque function to accept true wind and current velocities in the relative reference frame.

@patrick-5546 patrick-5546 added the sim Boat Simulator team label Mar 19, 2024
@DFriend01 DFriend01 requested a review from eomielan as a code owner April 6, 2024 00:43
@eomielan eomielan merged commit f729327 into main Jun 10, 2024
11 checks passed
@eomielan eomielan deleted the eomielan/refactor-boat-state branch June 10, 2024 01:44
samdai01 pushed a commit that referenced this pull request Jun 15, 2024
* Migrate PR 74 for boat_simulator

* Add arguments for rudder angle and trim tab angle

---------

Co-authored-by: Patrick Creighton <pcreighton429@gmail.com>
Co-authored-by: Eric Omielan <112828024+eomielan@users.noreply.github.com>
samdai01 added a commit that referenced this pull request Jul 13, 2024
* test

* minor changes

* added more comments for clarifications

* Minor additions and corrections

* Update custom_interfaces

* working through AIS parsing

* finished impelementing AIS parsing

* AIS parsing works according to tests, modified custom interfaces HelperAISShip by changing its id from int32 to uint32

* modified HelperAISShips

* cleaning up comments

* Implemented rounding

* Revert changes to website package-lock.json

* Completed

* Cleaning up some comments

* Fixed publishing boat sim inputs

* Modified battery frame and changed canid as needed for remaining frames. Unsure what what to do about sail cmd frames.

* cleaned up old can frames, ready for new ones

* Revert "cleaned up old can frames, ready for new ones"

This reverts commit ddf0a9f.

* Revert "Modified battery frame and changed canid as needed for remaining frames. Unsure what what to do about sail cmd frames."

This reverts commit d4e2a4f.

* Implement Fluid Generation (#328)

* Migrate code for issue 304

* test 1

* test 2

* Fix linter errors, remove TODOs, add test for next()

* Revert "test 2"

This reverts commit 6204043.

---------

Co-authored-by: Devon Friend <devon.fri.2000@gmail.com>
Co-authored-by: Eric Omielan <112828024+eomielan@users.noreply.github.com>
Co-authored-by: eomielan <eomielan@gmail.com>
Co-authored-by: Patrick Creighton <pcreighton429@gmail.com>

* Fixed DB Polling Lag (#395)

* Fixed DB Polling Lag

* No need for filter

* Fix warning from test_controllers unit test (#397)

* Refactor BoatState class for integration (#312)

* Migrate PR 74 for boat_simulator

* Add arguments for rudder angle and trim tab angle

---------

Co-authored-by: Patrick Creighton <pcreighton429@gmail.com>
Co-authored-by: Eric Omielan <112828024+eomielan@users.noreply.github.com>

* Add instance variables to physics node (#403)

* attempt to remove website changes

* Revert "attempt to remove website changes"

This reverts commit 396ad11.

* Update raw_width in can_frame_parser.cpp

* Updated more raw_width to uint16 in can_frame_parser.cpp

---------

Co-authored-by: hhenry01 <henryhuang2001@gmail.com>
Co-authored-by: Patrick Creighton <pcreighton429@gmail.com>
Co-authored-by: Aryan Roshan <arynrosh@gmail.com>
Co-authored-by: Devon Friend <devon.fri.2000@gmail.com>
Co-authored-by: Eric Omielan <112828024+eomielan@users.noreply.github.com>
Co-authored-by: eomielan <eomielan@gmail.com>
Co-authored-by: Jordan Chen <122140935+JordanChen123@users.noreply.github.com>
Co-authored-by: Devon Friend <69879126+DFriend01@users.noreply.github.com>
Co-authored-by: Sam Dai <daisam1215@gmail.com>
Co-authored-by: Sam Dai <89489298+samdai01@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sim Boat Simulator team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants