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

Telemetry Extractor delimiter on last item #1136

Closed
keck-in-space opened this issue Mar 18, 2020 · 0 comments · Fixed by #1143
Closed

Telemetry Extractor delimiter on last item #1136

keck-in-space opened this issue Mar 18, 2020 · 0 comments · Fixed by #1143
Milestone

Comments

@keck-in-space
Copy link
Contributor

keck-in-space commented Mar 18, 2020

Describe the bug
When using Telemetry Extractor with the following configuration, a delimiter is placed after the final item on the first row. This is an issue with tools like SQLite's importer function. SQLite expects one more item to exist due to the final comma, and since it doesn't there's a null column appended to each item.

DELIMITER ","
DONT_OUTPUT_FILENAMES

Note: the examples here do not have a comma on the last item of the first row.

To Reproduce
Steps to reproduce the behavior:

  1. Run Telemetry Extractor with the above configuration.
  2. Extract a packet.
  3. View the resulting file and note the final item also has a comma at the end of it.

Expected behavior
The final item of the first row shouldn't have a comma.

Environment (please complete the following information):

  • OS: Windows 10
  • COSMOS Version 4.4.2
keck-in-space added a commit to keck-in-space/COSMOS that referenced this issue Mar 18, 2020
@ghost ghost added the maintenance label Mar 19, 2020
@ghost ghost mentioned this issue Mar 25, 2020
@ghost ghost closed this as completed in #1143 Mar 26, 2020
@ghost ghost added this to the v4.5.0 milestone Nov 6, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant