Feature/signalhound - This is preliminary work for a Spike Signal Hound converter.#136
Open
KelseyCreekSoftware wants to merge 2 commits intosigmf:mainfrom
Open
Feature/signalhound - This is preliminary work for a Spike Signal Hound converter.#136KelseyCreekSoftware wants to merge 2 commits intosigmf:mainfrom
KelseyCreekSoftware wants to merge 2 commits intosigmf:mainfrom
Conversation
Implement converter for Signal Hound files to SigMF format with metadata extraction and IQ data handling.
Integrate Signal Hound file conversion into the main script, updating magic byte checks and handling for new file type.
5 tasks
Collaborator
|
I'll assume this isn't quite ready yet, but I'm keeping an eye on it. |
Teque5
reviewed
Feb 25, 2026
| from datetime import datetime, timezone, timedelta | ||
|
|
||
| import xml.etree.ElementTree as ET | ||
| from typing import Optional |
Collaborator
There was a problem hiding this comment.
Suggested change
| from typing import Optional | |
| from typing import Optional, List |
For backwards compat with python 3.7 (and to pass the build gate) you also need to change the line _parse_preview_track(text) -> List (capital L)
Teque5
reviewed
Feb 25, 2026
| "annotations": annotations, | ||
| } | ||
|
|
||
| return sigmf |
Collaborator
There was a problem hiding this comment.
This function should return SigMFFile object and not dict.
Teque5
reviewed
Feb 25, 2026
| DATATYPE = "ci16_le" # complex short int16 little-endian | ||
| # DATATYPE_SIZE = 4 # bytes per complex int16 sample (2 bytes I + 2 bytes Q) | ||
|
|
||
| logging.basicConfig( |
Collaborator
There was a problem hiding this comment.
A logger gets created by __main__.main() so I don't think we need this.
Teque5
reviewed
Feb 25, 2026
|
|
||
| # Define constants for Spike | ||
| ENDIANNESS = "<" | ||
| DATATYPE = "ci16_le" # complex short int16 little-endian |
Collaborator
There was a problem hiding this comment.
you appear to define this again later, so I think you can delete these globals
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is preliminary work for a Spike Signal Hound converter.
I attempted to use the SigMF File API as much as possible and follow the general form of the blue and wav converters.
Very limited testing, but archive, NCD and XML conversion seems to work.
One thing that needs to be validated is the handling of the IQ file data itself.
The Spike XML files contain a small CSV sample of the larger IQ file. Not sure what the best way to handle this "preview" data.
There are other TODO items called out in the code.