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

Readd support for reading in direct CSV exports from Radio Reference #547

Merged
merged 6 commits into from
Nov 29, 2021

Conversation

devicenull
Copy link
Contributor

The RadioReference.com talkgroup CSV format is not very different then the one trunk-recorder expects. This patch adds support for using the CSV exported directly from RadioReference, while still supporting the custom trunk-recorder format. It detects a RadioReference format file by looking for (and ignoring) the header line it inserts.

@robotastic
Copy link
Collaborator

This is a great idea! Could you include/attach a sample .csv file from radio reference, so I can test it out? I really have to upgrade sometime so I can download the .csv files.

@devicenull
Copy link
Contributor Author

Sure, sample attached.

demo.csv

@robotastic
Copy link
Collaborator

So - Slight wrinkle... in the talkgroups, the mode is a char variable. It can only work with a single character, so A, D, & E work, but TE is a problem. And I actually just checked my local system and they are using DE for encrypted channels. I will go work on changing it around so that mode is a string.

Cleaning up main.cc - Gain settings was merged in during an earlier PR.
@JoeGilkey
Copy link
Contributor

Performance wise, making the mode an enum would be better.

@robotastic
Copy link
Collaborator

Ok - I think I have things working. Merging in... thanks!
Can you check audioplayer.php? There was a conflict and I think I fixed things but may have messed it up.

@robotastic robotastic merged commit cf8ed83 into TrunkRecorder:master Nov 29, 2021
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.

3 participants