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

Drake/feature/additional sim uhf params #70

Merged
merged 7 commits into from
Aug 28, 2024

Conversation

DrakBoul
Copy link
Contributor

simulated_uhf.py:
Added additional simulated parameters to the simulated UHF like baud rate and mode. Also added functionality to get and set each parameter.

test_sim_uhf.sh
Created bash script for testing the simulated UHF using generic client program

README.md:
Added additional information about sending commands to the UHF.

@DrakBoul
Copy link
Contributor Author

The port that the simulated UHF currently uses for the GS is the port 1235. I want to change this to the official port found in the ex3_software/ex3_shared_libs/common/lib.rs but it has not yet been created. Would prefer this happens before merging this branch to main.

Copy link
Contributor

@rrasmuss4200 rrasmuss4200 left a comment

Choose a reason for hiding this comment

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

Awesome job! It seems to be working just like we want. I made a branch in ex3_software called incorporate_sim_uhf. It has the updated ports and stuff. 1805 and 1808 for coms and gs respectively I believe.

Next you can feel free to start working on getting it working with the rest of the architecture. I tried a little but got this error:

Exception in thread Thread-1 (receive_msg):
Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.10/threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "/home/rowan/AlbertaSat/ex3_simulated_subsystems/UHF/./simulated_uhf.py", line 282, in receive_msg
    print(f"Received: {message.decode('utf-8')}")
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc8 in position 4: invalid continuation byte

I don't know if they're related but bulk downlink didn't work either.
So feel free to make a branch off the one I created and merge this one to main.

@rrasmuss4200
Copy link
Contributor

I changed the port dw about it

@rrasmuss4200 rrasmuss4200 merged commit 99acaeb into main Aug 28, 2024
1 check passed
@DrakBoul DrakBoul deleted the drake/feature/additional_sim_uhf_params branch August 29, 2024 01:30
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.

2 participants