Skip to content

ADCS bug fixes 5 #276

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

ADCS bug fixes 5 #276

kmkar31 wants to merge 104 commits into from

Conversation

@kmkar31
Copy link
Collaborator

@kmkar31 kmkar31 commented Apr 22, 2025

Fixes faults in

  • ACS
  • MEKF

Akshat412 added 30 commits March 11, 2025 12:02
@Akshat412
Initial commit and requirements
8a3ce3e
@Akshat412
Basic Unix timestamp implementation from RTC
f88f060
@Akshat412
All applications use TPM
59df48a
@Akshat412
Removed unused import time in applications
2c405d9
@Akshat412
Removed temp RTC init in command task (I found a coin cell)
9757592
@Akshat412
Implemented state correction using commands
42c9d35
@Akshat412
TPM is now part of FSW core
b891120
@Akshat412
Trying to fix failing unit tests
d5164d0
@Akshat412
Trying to fix failing unit tests
1da7329
@Akshat412
Plans for time when RTC fails
f1d63dc
@Akshat412
Error handling in case RTC fails
a87f272
@Akshat412
Error handling in case RTC fails
3de5c24
@Akshat412
Update docs in TPM
463e20e
@Akshat412
Disabled precondition on uplinked time
22c22b3
@Akshat412
Resolved merge conflicts with main
023b630
@Akshat412
Segmented command task into functions
7ab2885
@Akshat412
GPS Task prints additional GPS debug info
15d73b8
@Akshat412
Merge conflicts
78eaf52
@Akshat412
More GPS task rework
4d0ad67
@Akshat412
Changed GPS logging and task frequency (for testing)
3209c04
@Akshat412
GPS time reference for TPM
5e6d913
@Akshat412
Removed GPS time fix update, needs more testing with GPS module
1c1170d
@Akshat412
Linter fixes
3622fef
@Akshat412
GPS task does nothing in STARTUP
a2cbe02
@Akshat412
Reset GPS task frequency back to 0.025 Hz
89141ce
@Akshat412
Updated TimeProcessor to TPM in apps and tasks
7e345bd
@Akshat412
Merge remote-tracking branch 'origin/main' into timing
42541ce
@Akshat412
SATELLITE.BOOTTIME used for boot time again
9b806ea
@Akshat412
Removed debug stating RTC is inactive in command task
26ba67f
@Akshat412
Removed time.time from pycubed.py
05c0f20
@kmkar31 kmkar31 linked an issue Apr 22, 2025 that may be closed by this pull request
ADCS: Fix everything #274
Open
@kmkar31
Copy link
Collaborator Author

kmkar31 commented Apr 22, 2025

Detumbling Fix:

  • Signs were wrong on coil allocations i.e., if +2.5V is provided to XP and XM coils, fields cancel out. Fixed this by assigning opposite signed voltages to coils on the same axis
  • spin stabilization gain was tuned to reduce detumbling time. Now detumbles in the range of 40 min to 2 hours

@Akshat412
Copy link
Collaborator

Akshat412 commented Apr 22, 2025

I think you should merge from main......

@kmkar31
Copy link
Collaborator Author

kmkar31 commented Apr 25, 2025 •
edited
Loading

All of ACS works
control_input
ing
omega
sun_point

@kmkar31 kmkar31 mentioned this pull request May 2, 2025
Open
@kmkar31 kmkar31 force-pushed the adcs-bug-fixes-5 branch from da75048 to 9abf093 Compare May 3, 2025 00:03
@kmkar31 kmkar31 force-pushed the adcs-bug-fixes-5 branch from 9abf093 to 2721cd3 Compare July 8, 2025 05:03
@kmkar31 kmkar31 force-pushed the adcs-bug-fixes-5 branch from 2447ea6 to 2a0bc37 Compare July 8, 2025 05:05
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.

ADCS: Fix everything

4 participants

@kmkar31 @Akshat412 @amaarquadri @Ibrassow