Skip to content

Nos3#850 Multiple GDS's#851

Merged
zlynch2 merged 8 commits into
devfrom
nos3#850
May 21, 2026
Merged

Nos3#850 Multiple GDS's#851
zlynch2 merged 8 commits into
devfrom
nos3#850

Conversation

@zlynch2

@zlynch2 zlynch2 commented May 19, 2026

Copy link
Copy Markdown
Collaborator

This PR is to add the capability of adding two ground stations into NOS3. This capabliy launches both yamcs and cosmos 4.5 together.

How to test?
Change the nos3 cfg to use the new multiple target build and run with cFS. Verify operations with both 1 GDS and 2 GDSs.

Submodule PRs and actions prior to closing this:
Cryptolib
Generic_radio

Closes #850

@zlynch2 zlynch2 requested a review from Donnie-Ice May 19, 2026 15:36
@codecov-commenter

codecov-commenter commented May 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.06%. Comparing base (6f2e4a2) to head (321da7e).
⚠️ Report is 23 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #851      +/-   ##
==========================================
- Coverage   72.75%   72.06%   -0.69%     
==========================================
  Files         761      761              
  Lines       90262    91123     +861     
  Branches     7137     7293     +156     
==========================================
  Hits        65668    65668              
- Misses      24594    25455     +861     

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

@zlynch2 zlynch2 changed the title Nos3#850 Nos3#850 Multiple GDS's May 19, 2026

@msuder msuder left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested both multiple GDS and single GDS and it works as expected. Reviewed code changes and they look ok.

@zlynch2 zlynch2 self-assigned this May 21, 2026
@zlynch2 zlynch2 merged commit b2d2b73 into dev May 21, 2026
3 checks passed
@zlynch2 zlynch2 deleted the nos3#850 branch June 11, 2026 15:28
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.

3 participants