nasa / nos3

NASA Operational Simulator for Small Satellites
Other
379 stars 86 forks source link

[nasa/Nos3#325 ] Equuleus upgrade #373

Closed rjbrown2 closed 1 month ago

rjbrown2 commented 1 month ago

Must review and approve these first:

rjbrown2 commented 1 month ago

Mark found that the nos3-mission.xml is still in sc-minimal-config.xml --- Change this back to sc-full-config.xml

msuder commented 1 month ago

Pulled the 325 branch. Updated submodules. Ran minimal and full scenarios and things worked as planned. Approved for merge.

jlucas9 commented 1 month ago

Looks good to me! Put in a minor hotfix to resolve 42 IPC conflicts with the thruster and IMU which seem to be from the dev branch and not this work in order to fully verify 42 launching and sun pointing. Confirmed CFDP class 2 also appears to work via test runner.

Just don't forget to close the required submodule merges and update at the nos3 / top level before merging this in!