-
-
Notifications
You must be signed in to change notification settings - Fork 201
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
player-url-i2s #24
Comments
I pushed the related correction to main. I also extended the example to demo how to debounce the input... |
in you example you need to change debouncer.debounce to nextBuffonDebouncer.debounce() I did some tests with player.next and I can not change between other channels than 0 en 1. |
Is this issue with my example ? |
I diden't set it right in the AudioSourceURL. I changed it from 5 to 2. I do the test now... and it works. |
I changed the constructor so that the array size is not needed any more to prevent this problem for others. |
Good, no I want to select a station over a2dp if this is possible. I am looking now for making this possible... |
Not sure if I understand your correctly but note the following: |
I have updated the library and now the player doesn't compile anymore...
|
Sorry about that, but I did't not get this when I tested in Arduino... |
Did you enabled the warnings for compiling? |
It's set to Default |
mine too... |
It is compiling now. Thx I tried to set the config file back to the settings that worked in the previous topic I have made. But now the music stops. Soo I tried different settings but it keeps chunking. |
Maybe it was the antenna again. I's running good now with the default settings |
Did you use the resolved links in your examples? |
I am using now a local http link out of your xml and it's running much better now. |
Could you please add a procedure "select radio station by index number" of the array url? |
I have made a station select. I will look to upload when I am back. |
have you tried the following in your sketch:
I did not want to provide the direct navigation on the player level because not all AudioSource objects can support this... |
I know, I did it the same way... It diden't cost me a lot of work to do, but I will take over the latest version. I also add this...
it says url 1 of 5 is playing... You could add this to your log if you like. |
If I want to make a portable radio I was thinking to impliment a small display... For that it will be usefull to have the name of the radio station in that array. You could also add the index to it. For now I will integrate it in to a existing build in spreaker system without display. But maybe I will make something like a portable in the futer. This is what I had in mind. |
I just wanted to highlight, that this is the advantage of my design: It is object oriented! Then you can just provide your own implementation as parameter to the AudioPlayer.... |
I don't know why this continuesly doesn't work with me and after revise it it does, but it doesn't change the station on index with me. The log says it is changing but nothing ever happend... |
This doesn't shows up
|
I have tested the url player with these external links.
The all work but when I run over the second time it crashes on this link below, strange because it worked well on the first time.
here is my stack description
What do you think it could be... I have to say I reased the stack to 68000, because I have tested it for my esp32 with sd card in my shetch and that works. |
many links are working but is it possible to support one of those
|
Most of these links are not working even if you open them in the browser. |
could they be merged in to one player? |
My decoder API is standardized. So all we would need is a combined decoder which determines the data format dynamically and forwards it to the correct implementation. Are you interested to pick up the challenge ? |
I am ready... You begin! |
I will not be able to work on this for the next couple of weeks. But I can give an outline what needs to be done with a couple of hints.
|
Since this is Windows, it's not in my scope. |
I m not interested too. My attention is now to the aac, m3u and mp3. |
I quickly double checked on github and did not find a single m3u parser in c or c++. update: we could build on vlc |
I don't have much for now because I cann't get the data to work with. Normally for the AudioDecoder I have to give an AudioType in libhelix, but I cann't do that. So I looking for a way to read the bytes from the stream without losing them.
|
I just committed a draft implementation. https://github.com/pschatzmann/arduino-audio-tools/blob/main/src/AudioCodecs/CodecHelix.h |
I diden't test the aac decoder yet... This need to be resolved first before we can continue to a general decoder. I went ahead testing the aac decoder and this is where it stops.
|
it's a difficuld issue, do you have engough information or just no have the time to sort this out? |
It is still on my to do list - but not very hight. I remember that AAC was working for me. |
Could be, I will look in to that. I just generated a list of belgium radio station. Here is the list.
|
Do do...
|
I am testing now with https://icecast.radio24.ch/radio24-rc-96-aac That I found in here https://bommag.ch/uploads/pdf/Internet-Radio%20direkte%20Streams.pdf I increased the #define STACK_SIZE to 60000 and #define AAC_MAX_FRAME_SIZE to 2500 , it diden't work It crashes somewhere in the class before it even starts to the setup(); Here is the exception error descriptor
I tried even this with the same results
If I increase
it stops crashing, and I get this.
but still no sound. |
Here is the complete log. (I have updated my log according to the next changes I have made)
|
Ooh, now I see I used this
and changed to this
but still no sound. it just stops copying after some time. I tried different links. All the same. |
if I comment this line #define USE_URLSTREAM_TASK, the message goos away, but than it starts crashing again. I don't use task in my own shetch for this test. I am out of leads... I am now testing again the mp3 links and I can tell it works beter without USE_URLSTREAM_TASK. Out of my experence I know that changing as less as possible gives the best results. There is soo much that can go wrong because the bigger file size. Could you please post your config file for the aac if you have an esp32? |
And is there something wrong in here?
What I don't understand is when I inspect a link in the browser I get audio/mpeg for the mp3 links, but we fill in audio/mp3 and it works. What do I need to fill in for the aac? |
I did a lookup in the code of the library and I diden't find an example for the aac. Can you make these work? I have only one that is working.
|
I give up. I have tried the whole day to make the aac work... but it diden't. Here is my last exception
I have xPortGetFreeHeapSize: 51516, soo I doubt it that could be it. And if is I doubt it, it cann't be done with an esp32. |
Just published some corrections in the arduino-libhelix library. |
The player-url-i2s works for changing from one to next. After that it stays on the second channel. It is trying to change to next but for one reason it keeps taking the second url.
The text was updated successfully, but these errors were encountered: