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

Remote Transceiver GlobalPath to Iridium - #269 #464

Draft
wants to merge 57 commits into
base: main
Choose a base branch
from
Draft

Remote Transceiver GlobalPath to Iridium - #269 #464

wants to merge 57 commits into from

Conversation

vaibhavambastha
Copy link

Description

Implemented functionality to transmit a series of global waypoints to Polaris via the Iridium satellite network. The changes include:

  1. Parsing incoming request data to extract JSON strings.
  2. Creating and populating a Polaris::GlobalPath object.
  3. Serializing the Polaris::GlobalPath object to a binary string.
  4. Using libcurl to make a POST request to the Rockblock server to transmit the serialized global waypoints.
  5. Handling the response from the Rockblock server, including managing specific error codes.
  6. Writing Iridium response information to the database.

A new table has been added to the database to store Iridium response logs. The schema follows the structure outlined in "Database Schemas → Iridium Response" on Confluence.

Verification

  • Ensured POST requests are correctly formatted and sent to the Rockblock server using libcurl.
  • Implemented and tested logging of Iridium responses into the database.
  • Testing performed with the virtual rockblock_web_server.py to simulate Rockblock server interactions, covering all specified error cases.
  • Verified that waypoints are correctly serialized and deserialized in testing using Google Protobuf and stored in database correctly.
  • Verified multiple global path POST requests were handled appropriately and stored in database correctly in test_remote_transceiver.cpp through tests TestPostGlobalPath and TestPostGlobalPathMult.

Resources

hsn200406 and others added 30 commits June 8, 2024 11:44
@hsn200406 @vaibhavambastha
rebasing updates transmit globalpath
2e3de00 
new_updates
@hsn200406 @vaibhavambastha
update2mar09
6e4c042
@hsn200406 @vaibhavambastha
update3
3ed3db3
@hsn200406 @vaibhavambastha
update4
9fdab53
@hsn200406 @vaibhavambastha
git_mar16
fba81b6
@hsn200406 @vaibhavambastha
2024-03-30, 1:41 pm
d391487
@vaibhavambastha
specs added
fcbab93
@vaibhavambastha
working on testing POST
395597f
@vaibhavambastha
added specs
373b840
@vaibhavambastha
header file modified
5a01d06
@vaibhavambastha
progress test POST
379b01f
@hsn200406 @vaibhavambastha
2024-03-30, 2:31 pm
571f1a6
@vaibhavambastha
need to do createPostBody for global path
66ec240
@vaibhavambastha
stuck on json
94705db
@vaibhavambastha
original test remote transceiver
6dc19df
@vaibhavambastha
Revert "specs added"
a8a684a 
This reverts commit fcbab93.
@vaibhavambastha
Merge remote-tracking branch 'refs/remotes/origin/transmit_globalpath…
481875c 
…' into transmit_globalpath
@vaibhavambastha
testing
13de662
@vaibhavambastha
test classes made
9e18a58
@vaibhavambastha
format of global path post body
d738e8a
@hsn200406
changes to post global path (thread)
ad59d2f
@vaibhavambastha
setting global waypoints from .data
c8a5e19
@hsn200406
changes to post global path
e90faa6
@vaibhavambastha
testmate executable working
c94e094
@hsn200406
Posting Http request via API reference Rockblock
823dc92
@vaibhavambastha
updated curl timeout
0556a6a
@hsn200406
remote transceiver rockblock fix
4621bf0
@vaibhavambastha
Done storeIridumResponse
17940dd
@vaibhavambastha
Completed verifyDBWrite for iridium response
144e584
@hsn200406
addition of http response body function
168a0e9
vaibhavambastha and others added 27 commits August 10, 2024 14:11
@vaibhavambastha
Storing iridium response in database
a560e87
@vaibhavambastha
Finished verifying iridium response
0c7e500
@vaibhavambastha
Merge branch 'main' into transmit_globalpath
1b76978
@vaibhavambastha
Fix mkTimestamp syntax
f16a32c
@vaibhavambastha
verifyDBWrite_GlobalPath added back
ce71764
@vaibhavambastha
Commited scripts and src
7751b42
@hsn200406
Remote transciever with expected response
fad666d
@vaibhavambastha
db done but converstion for pt node not working
9f845ad
@vaibhavambastha
test to verify iridium response in db passing
edf67a7
@hsn200406
Fixed function parameters for storing iridium response in the database
9b51cba
@vaibhavambastha
update parameters
125c778
@eomielan @vaibhavambastha
Simulator dimension mismatches & fluid generation config (#435)
9d46001 
* Fix issues in model and physics node

* Fix dimension mismatching

* Fix comments for constants

* Allow wind & current generation config

* Allow config of fluid cov in globals.yaml

* Prevent data collection from making file when not in use

* Fix formatting issues with README for ci

* Update fluid generation unit tests
@SPDonaghy @vaibhavambastha
Land Data Generator Script (#434)
f51866f 
* brought in changes from old branch

* updated pkg xml with required dependencies

* updated pkg xml and requirements

* added jupyter notebook, ignore array type in obstacles

* updated license info

* updated script to apply consistent 12.5NM buffer

* printing heads

* ended friendship with shapefile, friends with geojson now

* removed geopandas and fiona!

* some metadata changes to notebook

* implemented buffer amount, its set at 5000 meters right now

* updated generation to include buffer

* metadata

---------

Co-authored-by: Sean Donaghy <[email protected]>
@JordanChen123 @vaibhavambastha
production db setup (#437)
0763857
@lross03 @vaibhavambastha
Implemented power mode frame and ROS callbacks send data to CAN (#428)
004d4d3
@kristatraboulay @vaibhavambastha
Resolve sensor and ROS message naming conflict (#441)
a2a0330 
* Changed sensor names to resolve ROS conflict

* Updated WindSensor to SimWindSensor
@eomielan @vaibhavambastha
Resolve simulator unit test warnings (#446)
362f819
@FireBoyAJ24 @vaibhavambastha
Global Path Bug (#448)
ff40318 
* Made global path none in localpathstate and checked it when initializing reference_latlon

* Changed the test case to not have localpathstate to be none
@samdai01 @vaibhavambastha
refactor test rockblock web server to take in both data and error code (
f879895 
#450)

* refactor test rockblock server to intake waypoint data and error code

* add documentation for refactor of rockblock web server

* fix for linter
@SPDonaghy @vaibhavambastha
added pybind11 to dockerfile
c672778
@hsn200406
testing remote transsceiver with local host
952ef9f
@hsn200406
remote transciever changes for testing local host
420fead
@vaibhavambastha
writing global path data to tmp file
e23217a
@vaibhavambastha
iridium res verified in db
9ae8d6c
@vaibhavambastha
tested for multiple global path reqs passed
806f293
@vaibhavambastha
merge main into user/remote_trans branch
a9bef97
@vaibhavambastha
comments documentation done
fa49c78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samdai01 samdai01 Awaiting requested review from samdai01 samdai01 will be requested when the pull request is marked ready for review samdai01 is a code owner

@eomielan eomielan Awaiting requested review from eomielan eomielan will be requested when the pull request is marked ready for review eomielan is a code owner

@SPDonaghy SPDonaghy Awaiting requested review from SPDonaghy SPDonaghy will be requested when the pull request is marked ready for review SPDonaghy is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees

@samdai01 samdai01

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Transmit Global Path Flow
9 participants
@vaibhavambastha @samdai01 @hsn200406 @eomielan @SPDonaghy @JordanChen123 @lross03 @kristatraboulay @FireBoyAJ24