Skip to content

Conversation

@greglucas
Copy link
Collaborator

It can be at the closest point to the end of the parsing rather than within an intermediate generator. Add the CCSDS Header information for more helpful debugging if it is a known ccsds byte stream.

Checklist

  • Changes are fully implemented without dangling issues or TODO items
  • [n/a] Deprecated/superseded code is removed or marked with deprecation warning
  • [n/a] Current dependencies have been properly specified and old dependencies removed
  • New code/functionality has accompanying tests and any old tests have been updated to match any new assumptions
  • [n/a] The changelog.md has been updated

It can be at the closest point to the end of the parsing rather
than within an intermediate generator. Add the CCSDS Header information
for more helpful debugging if it is a known ccsds byte stream.
@greglucas greglucas requested a review from medley56 as a code owner August 4, 2025 02:25
Copy link
Member

@medley56 medley56 left a comment

Choose a reason for hiding this comment

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

Good suggestion. Merging.

@medley56 medley56 merged commit 5e8e913 into lasp:main Aug 4, 2025
2 checks passed
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.

2 participants