search

SpaceTeam / STS1_COBC_SW

Software for the communication and onboard computer (COBC) of SpaceTeamSat1 (STS1)
MIT License
10 stars 2 forks source link

Add read function for EPS sensors #258

Closed danielschloms closed 6 months ago

danielschloms commented 8 months ago

Description

This PR adds functionality to read the EPS sensors, including a test. What is left to do is testing this on the HW and discussing and choosing the final configuration for the averaging register.

Fixes #213

codecov-commenter commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.62%. Comparing base (623fc3f) to head (05eab43).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #258 +/- ## ========================================== + Coverage 87.11% 88.62% +1.51% ========================================== Files 25 25 Lines 613 686 +73 Branches 7 9 +2 ========================================== + Hits 534 608 +74 + Misses 79 78 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

danielschloms commented 7 months ago

This was tested by checking the battery voltage with the COBC V2.6 ID 3, and it looked correct.

PatrickKa commented 6 months ago

We need to do the HW tests again because we changed eps::Read() and eps::ReadAdc().

PatrickKa commented 6 months ago

I did the HW tests today and they worked.