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

Error After minDuration Transmission Removal #702

Closed
RestNPizza opened this issue Jun 13, 2022 · 4 comments
Closed

Error After minDuration Transmission Removal #702

RestNPizza opened this issue Jun 13, 2022 · 4 comments

Comments

@RestNPizza
Copy link

I have a couple of channels that I can't quite get the squelch right on that have a 0 to half a second static click sent out every so often. To combat this, I decided to implement the minDuration and minTransmissionDuration properties in my system section. When trunk recorder eliminates the transmissions that are below the minimum duration threshold, it is followed with an error saying No Transmissions were Recorded!

Correct me if I an wrong but I do not believe there should be an error that follows as the transmission not being recorded is the intended result.

Here's an example of the error:

[2022-06-13 00:53:13.006652] (info) [PCFD] 5C TG: 4 Freq: 154.175000 MHz Removing transmission less than 0.5 seconds. Actual length: 0.24725.
[2022-06-13 00:53:13.006742] (error) [PCFD] 5C TG: 4 Freq: 1.54175e+08 No Transmission were recorded!

Should the error be fixed to not generate in these instances?

Thanks

@tadscottsmith
Copy link
Contributor

You will get that error if the transmission that was removed was the only transmission for the call. However, that error also can happen if for some other reason (signal, etc.) no transmissions were recorded, so it's probably good to keep the error there.

@tadscottsmith
Copy link
Contributor

@robotastic I created a PR #735 that addresses this issue.

If no transmissions were recorded, it will still be logged as an error:

[2022-11-21 16:19:15.057531] (error)   [ramsey] 3034C   TG:      11700   Freq: 8.53112e+08      No Transmissions were recorded!

However, if all transmissions were removed because they did not meet the minimum transmission duration, it will be logged as informational:

[2022-11-21 16:22:18.073246] (info)   [ramsey]  3178C   TG:      11562   Freq: 8.52688e+08      No Transmissions were recorded! 2 tranmissions less than 0.35 seconds were removed.

@tadscottsmith
Copy link
Contributor

Just noticed I misspelled transmissions in "tranmissions less than" if you wanted to fix that up before committing. Thanks!

@tadscottsmith
Copy link
Contributor

@dgwood12 can you check this out and close accordingly?

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

No branches or pull requests

2 participants