deep5050 / radio-active

Play any radios around the globe right from the terminal :zap:
https://www.radio-browser.info/#/
MIT License
509 stars 25 forks source link

Error on record #72

Closed esturniolo closed 1 year ago

esturniolo commented 1 year ago

When you try to record a radio that transmit in another codec than mp3, it will fail. https://imgur.com/LDu7pbt

May be using -c:a libmp3lame in the ffmpeg record command will help. The previuos flag change the codec to mp3.

github-actions[bot] commented 1 year ago

Hi, @esturniolo, Thanks for your contribution :blue_heart: . Contributors :people_holding_hands: like you make the open source community :earth_africa: such an amazing place to learn :book: , inspire :angel:, and create :art: . We will investigate :eyes: and get back to you as soon as possible :+1: . Just make sure you have given us sufficient information :information_source:.

By that time enjoy this meme :point_down: , hope you like it :smile:

meme

Use this action on your projects. Use jokes on issues instead.

deep5050 commented 1 year ago

duplicate of #68 yes i have checked with libmp3lame it works as expected ๐ŸŽ‰ . Thank you so much for the trick!

will support on 2.7.1

deep5050 commented 1 year ago

hi, @esturniolo would you please try this insider version and give feedback about the recording feature? by default, it would record mp3,. if -filetype auto is provided it will try to detect the codec information from the URL and set the extension accordingly!

pip install git+https://github.com/deep5050/radio-active@2.8.0

esturniolo commented 1 year ago

I'll. If you can, please give me a few days.

I'm glad that the libmp3lame worked!

esturniolo commented 1 year ago

The recording feature now works as expected. Thanks!

But I've another question regarding the recording. I see that this new version doesn't show the UUID anymore, so how can I select the proper station to recording programatically?

deep5050 commented 1 year ago

actually, the table with UUID takes up a lot of space. I was planning to ignore it completely.

you can suggest me weather I should display UUID or not.

esturniolo commented 1 year ago

I understand, but I think that this info is needed in order to record something in the background. What about if you use on the last characters of the UUID in the screen, but pointing to the real one?

I mean you have this UUID 5517c22e-6453-11ee-8c99-0242ac120002

But in the screen you show:

โ”ƒ ID โ”ƒ Station                        โ”ƒ Country โ”ƒ UUID              |         Tags                     โ”ƒ
โ”กโ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ•‡โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”โ”ฉ
โ”‚ 1  โ”‚ Spoon                          โ”‚         โ”‚ 0242ac120002      |                                    โ”‚

If not, there is another way to record in the background?

deep5050 commented 1 year ago

We can show the UUID after we select a station. Or introduce an option to show detailed search results.

esturniolo commented 1 year ago

The idea to add an option to show the detailed view in resutls sounds great! So every user select what to see.