NASA-AMMOS / NASA-AMMOS/AIT-Core

Remove cmd CCSDS handling

Open
#367 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
56
Forks
35
Avg merge
1d 20h
Merged PRs (30d)
3

Description

Much like our telemetry dictionary handling in mentioned in #366, our command definitions support a ccsds flag indicating that commands should be wrapped in a CCSDS packet. It's rather implicit instead of explicit, poorly documented, and better handled via explicit command extensions or via server output stream handlers.

Let's drop this functionality.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No files or tests are named. Start by tracing where command definitions read the ccsds flag and where commands are wrapped in CCSDS packets, using the related telemetry-dictionary issue #366 for context. Done means the implicit CCSDS command handling is removed and the remaining command-extension or output-stream behavior is clear.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.