abdelaziz-mahdy / flutter_meedu_videoplayer

Cross-Platform Video Player for flutter
https://abdelaziz-mahdy.github.io/flutter_meedu_videoplayer/
MIT License
132 stars 69 forks source link

Abnormal playback of the new fvp version m3u8 #169

Closed AnonymHK closed 10 months ago

AnonymHK commented 10 months ago

[flutter_meedu_videoplayer] Exception: No Stream for textureId: -1. [flutter_meedu_videoplayer] #0 MdkVideoPlayer.videoEventsFor (package:fvp/src/video_player_mdk.dart:207:5)

1 VideoPlayerController.initialize (package:video_player/video_player.dart:488:10)

#2 MeeduPlayerController.setDataSource (package:flutter_meedu_videoplayer/src/controller.dart:533:7) #3 _M3u8ExamplePageState._setDataSource (package:flutter_meedu_videoplayer_example/pages/m3u8_page_example.dart:285:7) There is another issue, I am not sure if it is a problem with fvp. If the network is disconnected and restored, the buffered video will not continue to play!
abdelaziz-mahdy commented 10 months ago

I think it's related to fvp, yes it's from it's logic

Also on what platform?

I would love if you can open an issue for fvp explaining the steps to reproduce it

Just to help the package maintainer fix and improve the package

AnonymHK commented 10 months ago

Playing on the web is normal, but this error occurred on Android! c97b2999176131eb731362186cf461a

abdelaziz-mahdy commented 10 months ago

Just disable fvp on Android from the initmeedu

It will work using official video_player

AnonymHK commented 10 months ago

If there is a problem with even such a basic playback format, I am very concerned about the robustness of the fvp library!

AnonymHK commented 10 months ago

image Is fvp not used by default? Why would an error still occur?

abdelaziz-mahdy commented 10 months ago

Yes it's not used by default, but on the example I think I left it enabled

abdelaziz-mahdy commented 10 months ago

If there is a problem with even such a basic playback format, I am very concerned about the robustness of the fvp library!

I tested it using macos, and it was working very good,

Also it's in early stages so bugs are logical to occur, right now the only problem if windows or Linux has bugs those will be a problem

AnonymHK commented 10 months ago

I think it's related to fvp, yes it's from it's logic

Also on what platform?

I would love if you can open an issue for fvp explaining the steps to reproduce it

Just to help the package maintainer fix and improve the package

I am running M3u8ExamplePage in the example. You can try running it on an emulator or Android phone, and an error will be reported directly!

abdelaziz-mahdy commented 10 months ago

I think it's related to fvp, yes it's from it's logic

Also on what platform?

I would love if you can open an issue for fvp explaining the steps to reproduce it

Just to help the package maintainer fix and improve the package

I am running M3u8ExamplePage in the example. You can try running it on an emulator or Android phone, and an error will be reported directly!

But with fvp disabled that doesn't happen correct!!?

AnonymHK commented 10 months ago

Yes, after disabling, the playback of the m3u8 sample is normal!

abdelaziz-mahdy commented 10 months ago

opened an issue https://github.com/wang-bin/fvp/issues/15

AnonymHK commented 10 months ago

Thank you very much, you are still dealing with the problem so late!

abdelaziz-mahdy commented 10 months ago

Thank you very much, you are still dealing with the problem so late!

i had to suffer today to avoid making people suffer

hopefully we get the package back to working state

thank you for keeping up with me and helping when you had stuff to do <3

AnonymHK commented 10 months ago

I really like flutter_meedu_videoplayer and I look forward to more people using it! I hope it becomes more and more stable!

AnonymHK commented 10 months ago

I saw that the author of Fvp has released a new version that addresses playback issues in Windows and M3U8. Please update our version dependencies after testing!

abdelaziz-mahdy commented 10 months ago

I saw that the author of Fvp has released a new version that addresses playback issues in Windows and M3U8. Please update our version dependencies after testing!

they didnt release yet

abdelaziz-mahdy commented 10 months ago

they published, flutter upgrade should be enough

AnonymHK commented 10 months ago

Okay, waiting for your testing and updates!

abdelaziz-mahdy commented 10 months ago

Okay, waiting for your testing and updates!

i tested it on fvp it was working correctly, so everything should be good, let me know if there is another problem

since i cant test now

And a flutter upgrade will get the latest fvp for you to work with

AnonymHK commented 10 months ago

I just tested the new version, but unfortunately, I still cannot compile it!

FAILURE: Build failed with an exception.

CMake Error at CMakeLists.txt:67 (include): include could not find requested file:

  C:/Users/Pub/Cache/hosted/pub.flutter-io.cn/fvp-0.2.1/android/mdk-sdk/lib/cmake/FindMDK.cmake
AnonymHK commented 10 months ago

I have upgraded to 7.4.2 for Gradle !

abdelaziz-mahdy commented 10 months ago

can you do flutter clean and try again, i think the code failed to download the file, since i just tested and it works

AnonymHK commented 10 months ago

Flutter clean I have tried! Invalid, I checked the error message and found that the file FindMDK.cmake is not available in the directory fvp-0.2.1!

Cache/hosted/pub.flutter-io.cn/fvp-0.2.1/android/mdk-sdk/lib/cmake/FindMDK.cmake

android/mdk-sdk/ The directory 'mdk sdk/' does not exist

AnonymHK commented 10 months ago

I found the reason, https://sourceforge.net/projects/mdk-sdk/files/nightly/mdk-sdk-android.7z Not a valid download address, it's just a download page!

abdelaziz-mahdy commented 10 months ago

it does download for me

let me do a flutter clean and try again

edit: is still worked for me

can you delete the package folder in cache and try again?

AnonymHK commented 10 months ago

The actual download address is https://altushost-swe.dl.sourceforge.net/project/mdk-sdk/nightly/mdk-sdk-android.7z I have submitted an issue to the author for correction!

abdelaziz-mahdy commented 10 months ago

Well I hope it fixes the issue in your case, but for some reason it was working for me 😔

Let's see what the author will say