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

Only LSB of batteryChargingCurrent is sent in RTTY #8

Open
jgromes opened this issue Dec 9, 2019 · 0 comments
Open

Only LSB of batteryChargingCurrent is sent in RTTY #8

jgromes opened this issue Dec 9, 2019 · 0 comments
Labels
bug Something isn't working satellite Issue related to satellite software

Comments

@jgromes
Copy link
Contributor

jgromes commented Dec 9, 2019

Due to a cast from int16_t to uint8_t, only LSB of charging current is sent during RTTY system info transmission:

Communication_RTTY_Print_Hex(batteryChragingCurrent);

Also, current RTTY system info prints out letter shift symbol at the end of each hex-encoded byte. This makes transmission longer than it needs to be (although easier to decode in some cases).

@jgromes jgromes added bug Something isn't working satellite Issue related to satellite software labels Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working satellite Issue related to satellite software
Projects
None yet
Development

No branches or pull requests

1 participant