Introduction of downlinkHandling module to communication architecture#1323
Conversation
f6af0cf to
3be776c
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3be776cccd
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
5ba35e8 to
edec469
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: edec469cda
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
72e48b4 to
76f68ab
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 76f68abd32
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
76f68ab to
db4f933
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e2ed86a9fc
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
e2ed86a to
48e772d
Compare
schaubh
left a comment
There was a problem hiding this comment.
Nice PR. Even Codex commented on how detailed your RST documentation is ;-)
Just saw a few minor things, should be quick to address.
48e772d to
44caffa
Compare
bsk-xxxx)Description
This PR introduces the
downlinkHandlingcommunication module and all supporting interfaces, tests, and documentation required to use and validate it in Basilisk.The module maps link quality from
linkBudgetinto usable downlink outcomes, including:CNR1/CNR2and antenna RX stateThe current analytical BER path uses the standard coherent BPSK/QPSK-style AWGN BER expression. User-selectable modulation/coding models and empirical BER lookup tables are intentionally left as future extensions.
Core implementation and integration work includes:
New downlink message payload and diagnostics fields:
src/architecture/msgPayloadDefC/DownlinkHandlingMsgPayload.hNew/hardened
downlinkHandlingmodule implementation and interface:src/simulation/communication/downlinkHandling/downlinkHandling.hsrc/simulation/communication/downlinkHandling/downlinkHandling.cppsrc/simulation/communication/downlinkHandling/downlinkHandling.isrc/simulation/communication/downlinkHandling/Custom.cmakeUnit test coverage:
src/simulation/communication/downlinkHandling/_UnitTest/test_downlinkHandling.pyDocumentation and figures:
src/simulation/communication/downlinkHandling/downlinkHandling.rstsrc/simulation/communication/downlinkHandling/_Documentation/Images/DownlinkHandlingFlow.pngsrc/simulation/communication/downlinkHandling/_Documentation/Images/DownlinkHandlingReliabilityChain.pngsrc/simulation/communication/downlinkHandling/_Documentation/Images/DownlinkHandlingFlow.svgsrc/simulation/communication/downlinkHandling/_Documentation/Images/DownlinkHandlingReliabilityChain.svgRelease note snippet:
docs/source/Support/bskReleaseNotesSnippets/downlinkHandling.rstCommit organization is by feature layer:
downlinkHandlingmodule and SWIG interfaceVerification
Validation commands for this branch:
cmake --build dist3 -j4.venv/bin/pytest -q -p no:rerunfailures src/simulation/communication/downlinkHandling/_UnitTest/test_downlinkHandling.pyTest coverage includes:
REMOVE_ATTEMPTEDvsREMOVE_DELIVERED_ONLY)CNR1/CNR2, antenna RX states)Documentation
Documentation delivered in:
src/simulation/communication/downlinkHandling/downlinkHandling.rstIt includes:
C/N0,Eb/N0, BER, PER, retry-limited ARQ, throughput scaling)spaceToGroundTransmitteranddownlinkHandlingshould not be used as competing downlink removers on the same storage partitionsFuture work