Skip to content

830 fix reaction wheel and thruster commanding to be spacecraft specific#831

Open
msuder wants to merge 2 commits intomainfrom
830-fix-reaction-wheel-and-thruster-commanding-to-be-spacecraft-specific
Open

830 fix reaction wheel and thruster commanding to be spacecraft specific#831
msuder wants to merge 2 commits intomainfrom
830-fix-reaction-wheel-and-thruster-commanding-to-be-spacecraft-specific

Conversation

@msuder
Copy link
Contributor

@msuder msuder commented Feb 13, 2026

Summary of PR goals.

How to test?

  • Change file cfg/sims/nos3-simulator.xml <spacecraft> tag from 0 to 1 for the GENERICRWSIMDATA42SOCKETPROVIDER
  • make -j to build and get the nos3-simulator.xml file into place
  • Launch NOS3 and quickly attach gdb to 42.
    • Verify that when the SC transitions to mode 2 that the reaction wheel simulator window shows commands to spacecraft 1 reaction wheels.
    • Verify that 42 gets commands for spacecraft 1 reaction wheels at TxRxIPC.c:682.
    • Note that this will cause 42 to seg fault since there is not actually a spacecraft 1 and so we try to index a spacecraft array with an index that does not exist.
  • Revert the change to nos3-simulator.xml
  • Change file nos3-simulator.xml <spacecraft> tag from 0 to 1 for the GENERIC_THRUSTER_42_PROVIDER
  • make -j to build and get the nos3-simulator.xml file into place
  • Launch NOS3 and attach gdb to 42. Verify that you can enable the thrusters and send a thruster command from COSMOS and that it is for spacecraft 1.
    • Do this by viewing the generic thruster sim output window and by seeing in gdb the command sent for spacecraft 1 in TxRxIPC.c:698.
    • Note that 42 will segfault if you proceed since there is no SC[1].

Submodule PRs and actions prior to closing this:

Closes #830 .

… to use config file specified spacecraft number in generic_reaction_wheel commanding.
@msuder msuder requested a review from zlynch2 February 13, 2026 16:49
@msuder msuder self-assigned this Feb 13, 2026
@msuder msuder linked an issue Feb 13, 2026 that may be closed by this pull request
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.74%. Comparing base (842aa0e) to head (ad4e0db).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #831   +/-   ##
=======================================
  Coverage   72.74%   72.74%           
=======================================
  Files         761      761           
  Lines       90262    90262           
  Branches     7133     7135    +2     
=======================================
  Hits        65664    65664           
  Misses      24598    24598           

☔ 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.

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.

Fix Reaction Wheel and Thruster Commanding to be Spacecraft Specific

2 participants