search

CCSDSPy / ccsdspy

I/O interface and utilities for CCSDS binary spacecraft data in Python. Library used in flight missions at NASA, NOAA, and SWRI
https://ccsdspy.org
BSD 3-Clause "New" or "Revised" License
74 stars 18 forks source link

adds documentation on CSV loading #114

Closed jmbhughes closed 4 months ago

jmbhughes commented 4 months ago

This resolves #64 by adding a new documentation page that outlines the CSV format required for loading packets.

I'm open to any suggestions to improve clarity.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.05%. Comparing base (f3957b6) to head (7d11be5). Report is 4 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #114 +/- ## ========================================== + Coverage 95.75% 96.05% +0.29% ========================================== Files 7 7 Lines 707 710 +3 ========================================== + Hits 677 682 +5 + Misses 30 28 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ddasilva commented 4 months ago

Thanks for submitting @jmbhughes ! I'll take a look at this over the weekend.

ddasilva commented 4 months ago

Thanks so much @jmbhughes! I think this looks pretty good. I made some suggestions to improve readability of the documentation. Having this page on the website will be good motivation for us to make the CSV format more feature complete (e.g., #115).

jmbhughes commented 4 months ago

These changes all look good to me. I'll make them ASAP.

jmbhughes commented 4 months ago

@ddasilva I updated with the suggested changes, so it's good to go.

ddasilva commented 4 months ago

Merged. Thanks @jmbhughes !