Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvements to XTCE generator script(s) #576

Closed
bourque opened this issue Jun 4, 2024 · 0 comments · Fixed by #680
Closed

Improvements to XTCE generator script(s) #576

bourque opened this issue Jun 4, 2024 · 0 comments · Fixed by #680
Assignees
Labels
enhancement New feature or request Student Tasks suitable for student work

Comments

@bourque
Copy link
Collaborator

bourque commented Jun 4, 2024

We have scripts to generate XTCE-formatted XML files for each instrument (example here). These are used to generate the "packet definition files" which are used to decommutate packet data from the spacecraft.

There are some improvements we could make to these scripts:

  • Make a single, generalized script that can process any instrument, instead of having a script for each instrument. This could be done by passing in command line argument(s) for the instrument to process, the path to the user's telemetry spreadsheet, the path to which to write the packet definition file, etc.
  • Improve the information that gets printed to the terminal for the user. Currently it is pretty bare bones.
  • Add appropriate unit tests
@bourque bourque added enhancement New feature or request Student Tasks suitable for student work labels Jun 4, 2024
@bourque bourque added this to IMAP Jun 4, 2024
@anamanica anamanica self-assigned this Jun 28, 2024
@github-project-automation github-project-automation bot moved this to Done in IMAP Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Student Tasks suitable for student work
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants