NASA-AMMOS / NASA-AMMOS/AIT-Core
Remove tlm CCSDS handling
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 56
- Forks
- 35
- Avg merge
- 1d 20h
- Merged PRs (30d)
- 3
Description
Packets can currently be marked as "CCSDS Packets" in our dictionary. This, effectively, just results in an injection of CCSDS fields into the packet definition. It's rather implicit instead of explicit, poorly (not?) documented, and easily replicated via include-ing a CCSDS header definition (see our example dictionary where we do exactly this). It's also not very user-friendly since the field names are defined in tlm.py and they're not necessarily intuitive. Having to search through a source file to figure out that the secondary header field is called shflag is annoying.
Let's drop this functionality
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the CCSDS handling in tlm.py and compare it with the example dictionary, which explicitly includes a CCSDS header definition. Trace how a dictionary entry marked as "CCSDS Packets" injects fields, then remove that implicit path while preserving the documented include-based approach. Done means the marker no longer adds CCSDS fields and the example remains usable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100