-
Notifications
You must be signed in to change notification settings - Fork 24
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
A Few Issues #31
Comments
The example works for me, and for another person who tested it. Could you cut & paste the relevant parts of the error message you're seeing?
I can't immediately think of a reason why this would happen. Can you provide any more details on the symptoms? Does the TCP port still accept input?
This should be fixed by c13100c. There was no need to add extra parameters to the block, since the Layer 2 encoder already receives program type information from the SIS & SIG encoder block in the SIG packet. |
The album art I have since gotten to work after some changes in my code. It does have some issues with corruption, but I think I'm sending it to the TCP port too quickly before it is fully written to disk. In terms of the HD Radio metadata no longer working around 24 hours, the issue is that after exactly 12 hours 0 minutes and 0 seconds, the TCP port stops accepting input for the metadata commands, despite my script sending it to it detecting that the metadata is connected. After this 12 hour mark, the TCP port stops accepting input and acts normal in my script, but doesn't actually register anything to the program. In terms of the issues with the PTYs, they have since been fixed and that was a rather fast fix for this project. Thank you a lot. |
To be clear, you're talking about the That it fails after precisely 12 hours is an interesting clue, but I still haven't thought of a reason why that might happen. Perhaps I'll add some additional debug logging to the blocks to help with diagnosis. |
Yep. I am talking about the artist, title and lot commands. To be exact, the album art also ceases working after the 12 hour mark. This has happened at least 2 times, if not 3 times, since I have started using this newer version of gr-nrsc5. It might be an issue with the TCP PDU on gnuradio, if it is determined to be an issue with gnuradio, one of us might have to go and post an issue on their repo. But, it might be an issue with configuration or gr-nrsc5, so I don't think we can rule it to gnuradio entirely just yet. |
After extensive testing and playing around over the past week I have patched the issue myself, or what I feel as if has patched the issue. I have been going strong with album art and radio text for around 24 hours now. I have detailed what I did to fix it in my pull request which can be found here that fixes the issue. #32 |
I will still be doing testing for a few more days just to completely confirm this was the issue and that nothing else was changed on my system during testing to make it have a longer delay. I will write back here in 24-48 hours if the script is still working. |
Testing has finished, issue was resolved with my pull request. |
As far as I can tell, all issues reported here have now been fixed, so I'll close off this issue. Let me know if I've missed anything. |
I'm extremely, extremely happy with the current state of the project. Everything is now functional, something that I am extremely happy with and have been playing around with for the past 5 days.
I do have a few bug reports I'd like to mention. The first bug report is that the HD Radio Album Art sending python example is nonfunctional. I'd like the ability to be able to pipe in album art from an URL, and I wasn't able to get the software to send a 200x200 JPEG or PNG that was saved to the folder containing all the other AAS info AFTER the transmitter was started. It would just output a massive error with a bunch of binary, something that'd be impossible for me to screenshot unless there was an absolute need for screenshotting the entire error.
Something else that I've noticed when running gr-nrsc5 24/7 for the past 4-5 days is that the HD Radio metadata eventually crashes out and stops working around the 24 hour mark of the transmitter running. It won't allow anymore metadata updates and will refuse any additional artist or title submitted to the network sockets.
Another issue is that there is some weird behavior with the PTYs on certain receivers. On NRSC5, it decodes my station with the correct PTYs as seen in the attachment below.
![image](https://private-user-images.githubusercontent.com/40074095/257234733-e7cbd606-0fc3-4605-8075-4a864f3b3720.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTkxMDk3MTgsIm5iZiI6MTcxOTEwOTQxOCwicGF0aCI6Ii80MDA3NDA5NS8yNTcyMzQ3MzMtZTdjYmQ2MDYtMGZjMy00NjA1LTgwNzUtNGE4NjRmM2IzNzIwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA2MjMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNjIzVDAyMjMzOFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQwNzgwNWEwMjY4YTg0NjM4MDRlMTNjMTEwYjhiMWJkODQ3MTUxYWJjZTIwMWQxZTc3NzYxYTNiOTZhZDdhMjYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.s_CGOYV2IyjifZuIhWjQkFL4e8KB_znyNoTOjiy4Dzk)
However, when locking the station on a Sangean HDR-14, PTYs simply display as "[No PTY]" on the receiver. I'm nearly certain that I've figured out where the problem resides in the code, however due to unfamiliarity with python bindings (I know how to implement it in C++, just not in the adjacent Python bindings). See
gr-nrsc5/lib/l2_encoder_impl.cc
Line 221 in 92e56c0
Would allow the HDR-14 to be able to decode program types. I'm unsure exactly how to implement this in the current version because without having decent python experience, I'm not able to adequately test and implement this feature myself. I tried making the L2 encoder have the same blocks as the SIS encoder, but had no luck due to the python bindings not matching up.
Hopefully you can fix the issue with the PTYs by implementing it into the L2 encoder alongside the already implemented program types in the SIS encoder.
Thank you for continuing this project. Wish you the best.
The text was updated successfully, but these errors were encountered: