Skip to content

Detumble #301

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
Draft
wants to merge 34 commits into
base: main
Choose a base branch
from
Draft

Detumble #301

wants to merge 34 commits into from

Conversation

@nateinaction
Copy link
Member

@nateinaction nateinaction commented Aug 11, 2025 •
edited
Loading

Summary

DRAFT

This PR adds a manager for the Proves V3 Magnetorquers as well as the required detumbling logic.

Still needs work:

  • Need to better understand the control flow
    • When do we stop using the magnetorquers?
    • What should the interface look like for starting/stopping magnetorquers
  • Actually manipulate the v3 magnetorquers, currently it's only scaffolded.

Previous work:
proveskit/solar_boards#4
https://github.com/BroncoSpace-Lab/BroncoSat-1-Flight-Computer/blob/master/ADCSC.py

How was this tested

  • Added new unit tests
  • Ran code on hardware (screenshots are helpful)
  • Other (Please describe)

nateinaction and others added 30 commits July 17, 2025 18:24
@nateinaction
Add sensor readings module
f930ae2
@nateinaction
Add errors
c1aee3f
@nateinaction
Merge branch 'main' of github.com:proveskit/circuitpy_flight_software…
d6c5218 
… into sensor-readings
@nateinaction
Add async support to lis2mdl manager
8965c8a
@nateinaction
Error tests working
f819f8e
@nateinaction
Fix test
98fa236
@nateinaction
Merge branch 'main' of github.com:proveskit/circuitpy_flight_software…
9afdde8 
… into sensor-readings
@nateinaction
All managers using sensor readings
79c0074
@nateinaction
Fix lsm6dsox and ina219 tests
0a34f63
@nateinaction
A few changes
7c83a70
@nateinaction
Fix tests
d524dac
@nateinaction
Fix tests
8b715a1
@nateinaction
remove accidentally committed files
c2853e8
@nateinaction @Copilot
Update tests/unit/sensor_reading/test_gyro.py
b01b726 
Co-authored-by: Copilot <[email protected]>
@nateinaction @Copilot
Update tests/unit/sensor_reading/test_acceleration.py
3059580 
Co-authored-by: Copilot <[email protected]>
@nateinaction @Copilot
Update pysquared/protos/reading.py
7affc6f 
Co-authored-by: Copilot <[email protected]>
@nateinaction @Copilot
Update pysquared/protos/reading.py
43eac66 
Co-authored-by: Copilot <[email protected]>
@nateinaction @Copilot
Update pysquared/protos/reading.py
db12bce 
Co-authored-by: Copilot <[email protected]>
@nateinaction @Copilot
Update tests/unit/sensor_reading/test_magnetic.py
6389f5c 
Co-authored-by: Copilot <[email protected]>
@nateinaction
Add coverage for exceptions handled in beacon
17b0de5
@nateinaction
Remove async work from lis2mdl
941c8eb
@nateinaction
Remove unused try/except in power_health
1ca9d54
@nateinaction
Refactor beacon to reduce cognitive complexity
142dff1
@nateinaction
Update naming for some measurements
0b6cc8c
@nateinaction
Fix renaming error
c517dbf
@nateinaction
Rewrite detumble
0f91230
@nateinaction
Scaffold tests
945b0ff
@nateinaction
Add divide by 0 checks
dec1f5e
@nateinaction
Improved comments
dfee616
@nateinaction
Add tests
ee49389
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
77.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@nateinaction nateinaction changed the base branch from main to sensor-readings August 11, 2025 05:54
Base automatically changed from sensor-readings to main August 20, 2025 04:52
@Mikefly123 Mikefly123 mentioned this pull request Aug 31, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@nateinaction