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

Implement Fluid Generation #328

Merged
merged 9 commits into from
Jun 1, 2024
Merged

Implement Fluid Generation #328

merged 9 commits into from
Jun 1, 2024

Conversation

arynrosh
Copy link
Contributor

@arynrosh arynrosh commented Mar 16, 2024

Description

Verification

  • Added tests to verify functionality of Velocity, Next, Speed and Distance classes.

@arynrosh arynrosh added the sim Boat Simulator team label Mar 16, 2024
@arynrosh arynrosh self-assigned this Mar 16, 2024
@DFriend01 DFriend01 requested a review from eomielan March 19, 2024 06:38
@arynrosh arynrosh requested a review from jahn18 as a code owner March 22, 2024 01:50
Copy link
Member

@eomielan eomielan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work so far! Just a couple things to address.

@eomielan eomielan assigned eomielan and unassigned eomielan Mar 24, 2024
@eomielan eomielan self-requested a review June 1, 2024 23:29
@eomielan eomielan merged commit 52e2e64 into main Jun 1, 2024
11 checks passed
@eomielan eomielan deleted the arynrosh/304-fluid-simulation branch June 1, 2024 23:42
samdai01 pushed a commit that referenced this pull request Jun 15, 2024
* 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>
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.

Create Fluid Generator class
4 participants