Skip to content

699 update 42 version 2#745

Merged
jlucas9 merged 8 commits intodevfrom
699-update-42-version-2
Jul 8, 2025
Merged

699 update 42 version 2#745
jlucas9 merged 8 commits intodevfrom
699-update-42-version-2

Conversation

@msuder
Copy link
Contributor

@msuder msuder commented Jun 17, 2025

msuder added 5 commits June 6, 2025 10:20
…pacecraft/COSMOS files to enable thrusters. Update generic CSS, EPS, FSS, IMU, magnetometer, reaction wheel, star tracker, thruster, torquer(and hwlib) and NovAtel OEM615 to use the latest 42 code.
@kevincbruce
Copy link
Contributor

Was testing and Science Mode didn't seem to run. Going to try again quickly.

@kevincbruce
Copy link
Contributor

Was testing and Science Mode didn't seem to run. Going to try again quickly.

Tried again. Science isn't triggering, and the sensors don't appear to be parsing the data correctly. Will look at it more once I'm in the office.

Copy link
Contributor

@kevincbruce kevincbruce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After a fix from Mark to get it pulling the right 42 version pre-merge, verified telemetry, sunpoint, and inertial mode, plus science working.

Copy link
Contributor

@Isgaroth-the-Green Isgaroth-the-Green left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have been having difficulty with the IMU; it is giving me values of 100 or so for most of the angular rates, but the 42 window does not show any movement (and turning the ADCS onto B-Dot does not change it at all). I also tried running the IMU tests, and they nearly all failed.

@kevincbruce
Copy link
Contributor

Reposting from chat:

Reviewers of issue 699/PR 745
When reviewing you need to edit the scripts/cfg/prepare.sh script and have the 42 clone line clone branch nos3-issue#699. Then do make uninstall and make prep

@jlucas9 jlucas9 dismissed Isgaroth-the-Green’s stale review July 8, 2025 11:28

Latest updates appear to resolve.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.65%. Comparing base (2f800dc) to head (400b5b9).
Report is 81 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #745      +/-   ##
==========================================
- Coverage   73.95%   73.65%   -0.31%     
==========================================
  Files         761      761              
  Lines       93579    93963     +384     
  Branches     6983     7062      +79     
==========================================
- Hits        69207    69206       -1     
- Misses      24372    24757     +385     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jlucas9 jlucas9 merged commit 30a95bf into dev Jul 8, 2025
3 checks passed
@jlucas9 jlucas9 deleted the 699-update-42-version-2 branch July 8, 2025 11:56
vedantt21 pushed a commit to vedantt21/nos3-mantle that referenced this pull request Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Comments