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

Determine why BattRT is wrong in DetailedStatus #201

Open
1 task
zCoCo opened this issue Mar 1, 2023 · 0 comments
Open
1 task

Determine why BattRT is wrong in DetailedStatus #201

zCoCo opened this issue Mar 1, 2023 · 0 comments
Assignees
Labels
bug Something isn't working bug-min Likely inconsequential that shouldn't affect any aspect of mission, even nominal ones. small Only several lines of code and quick to test. WD Code changes to the WatchDog
Milestone

Comments

@zCoCo
Copy link
Member

zCoCo commented Mar 1, 2023

For some reason the thermistor data in the WD's DetailedStatus packets are wrong (always 1), specifically BattRT (the battery thermistor). No idea why. Data comes through just fine in the heartbeats. Could suggest a broader formatting issue with DetailedStatus but bit-boundaries in the packet have been thoroughly tested and all the other fields have change when and how they are expected to in all other testing.

Low priority because we already have this data coming down in heartbeats.

NOTE from #196 :
Plugged a 5k resistor (i.e. 25C or 298K) into RT port. Heating status changed to NOT_HEATING (suggesting that the WD sees a new ADC value) but Tbatt in DetailedStatus stayed at 1, suggesting a data packing issue. BattTemp in Heartbeat and in Hercules telem incorrectly reported charging thermistor instead of main battery thermistor, so it's possible this is just a detailed status data packing issue.

See #206 . Might be related...

  • Thoroughly re-test all DetailedStatus downlink fields before merging if any changes were made to how it's packed.
@zCoCo zCoCo added bug Something isn't working small Only several lines of code and quick to test. WD Code changes to the WatchDog bug-min Likely inconsequential that shouldn't affect any aspect of mission, even nominal ones. labels Mar 1, 2023
@zCoCo zCoCo added this to the RC9 milestone Mar 1, 2023
@zCoCo zCoCo self-assigned this Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working bug-min Likely inconsequential that shouldn't affect any aspect of mission, even nominal ones. small Only several lines of code and quick to test. WD Code changes to the WatchDog
Projects
None yet
Development

No branches or pull requests

1 participant