-
Notifications
You must be signed in to change notification settings - Fork 751
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
Buffering / Stream support #353
Comments
@stavros-zavrakas use react-native-video for streaming for now. works with audio streams too :) |
That's a pretty sucky response to what is clearly a needed feature. Why |
OK, sorry. I didn't explain because I felt that the issue had been discussed in other topics. The deal is that this library doesn't use a library that supports streaming #292. We would have to rewrite and use the correct library. I don't know the amount of work required, but I bet it's not trivial. If someone wants to tackle that, great, but I'm not aware of anyone so far. If someone wants to submit a PR with the correct AVPLayer, I'm all game for that. |
@johncblandii @trepidity I agree that supporting streaming is not easy because it requires lots of work in native side (almost equivalent to re-writing this library). Since this library is used by production apps including our team's, I guess we need to focus on stabilization of the existing functionality. Currently, too many issues open, so this library is getting messy (see the pull requests which hasn't been processed for a long time). |
@trepidity that's clearer. If I get enough free time, I'll push a PR. @ohtangza feel free to focus on stabilization. Just when network playback becomes a feature the natural next step is streaming audio (either via progressive download or otherwise). Labeling it as help wanted is a great step forward to getting the help from those of us in the community capable of such a rewrite. |
@johncblandii Thanks for your prompt input on this. As you said, we'd like to make it clear somewhere in docs about the directions. Please understand that this is not a judgment on whether the feature is important or not. It's just not a trivial work because we need to change the core part of this library as @trepidity commented. If you submit a pull request to implement streaming, my team would be happy to review it because we could enhance our user's experience to a certain degree :) In the meantime, |
No problem, @ohtangza. I get where you guys are coming from for sure. |
Regarding |
If you are looking for other options, there is also https://github.com/react-native-kit/react-native-track-player which supports streaming |
Curious to know how this performed for everybody? |
Is there a plan to support buffering / stream support?
I've tried to use the sound from the network but I wasn't successful. There is this module (https://github.com/tlenclos/react-native-audio-streaming) that is not really maintained anymore but it is able to stream from the network. Do you think that this can be supported at some point in react-native-sound?
The text was updated successfully, but these errors were encountered: