-
Notifications
You must be signed in to change notification settings - Fork 19
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
Ads playing crashing the app #277
Comments
I don't know why the tool will not work. If you can share the logs will help. There is s related issue |
I have the same problem on my Q8FN but not on my Q60R. I am surprised that i even get ads when i am subscribed to a channel. Did twitch change something on their API or why are there suddenly ads showing? |
the app doesn't have access to your twitch sub or turbo privileged. you can see what the app has here https://github.com/fgl27/smarttv-twitch#twitchtv-app-for-samsung-tv-authorization-request-page twitch sometimes doesn't show ads for some reason now they show to more people. No "me too" please, one person reporting an issue is fine, we need someone to help debug and not add to the complaining. |
well you say it as if i wouldnt have provided further information by telling where the problem occurs and where not. i could try to help with debugging but i am not familiar on how to debug samsung tv apps. |
Here is the how to log |
Hello @fgl27 . I managed to debug the application when crashing while playing ads (used Chromium 74 because "--disable-web-security and "--disable-site-isolation-trials" flags wasn't working for me). Here is the screenshot when the crash occurs: And the code in the debugger i that column: Hope it helps to find the problem here. Dont heasiate to ask If you want me to try something. |
It does help, but not to resolve. that crash is outside the app code, on the Samsung avplayer API, that the app doesn't control. @maeseoki can you run the test again but this time install not from release? If yes install it by downloading the repo From https://github.com/fgl27/smarttv-twitch/archive/refs/heads/master.zip Or from the main page... code > download zip and them here https://github.com/fgl27/smarttv-twitch#setup-the-application import this folder https://github.com/fgl27/smarttv-twitch/tree/master/app and then when the error occurred before screenshotting open these triangular buttons like this, so we can see the full path it takes to crash Maybe with this extra info, we will understand this better and help. Thanks for the help. |
I am currently trying to get the debug app to work on my TV as OP seems to be missing. I had to setup Tizen Studio etc from scratch, and I have imported the "app" folder from the master branch. I deleted the Twitch app from the TV. When I try to run it I get this error, and it is so unspecified I dont even know where to start. Please advice |
That is a Tizen SDK bug. On windows it has various problems, Samsung doesn't properly support it. As is an java error maybe changing the java version helps but maybe not. So try a old version of java to see if that helps. I use Ubuntu and is working fine, but the last time I update Tizen was months ago. So if someone try that maybe buged also. |
Thanks for the quick reply, I will find a spare machine here to setup with Ubuntu tonight, and then hopefully get some debugging done. |
I have my spare laptop setup and I finished configuring the Tizen studio, with certificates etc. I could finally reupload the app to the TV successfully using the 4.0.1_V9.zip file I wanted to test Tizen studio using that file as I cannot figure out any way to get past the error I get when trying to install the from the "app" folder in the master.zip. Is there something I need to do before I can use the master.zip to install the debug version? Any extra steps etc? I get the same error as above for the master.zip method to try to get debug version installed. My TV is Tizen 3.0, if that has any implications |
No diff the install process is the same, the normal mode and debug must give you the same error. does? |
I can install and run both modes from the 4.0.1_V9.zip. I get no output in debug mode from running it from this file. I cannot install or run any of the modes from importing the app folder in the master.zip. Not sure how to proceed. Should I get output from the app running it in debug when using the files from 4.0.1_V9.zip ? |
I got it running, apparently I needed to build a signed package first, I did not realize it wouldnt just do it itself automatically. I got the master.zip version running now, but debug on my ubuntu machine only shows a blank white page. Suggestions of how to get them to talk together? |
None, let me see it |
What browser is that? Chrome always works for me, but you need to defined it as default browser. |
It is the latest chrome, and it is configured in the studio. Problem is it comes up blank. I tried googling, and the answer seemed to be to try older versions, which I did. I tried different versions of both Chrome and Chromium, unfortunately none worked correctly, so I'm stuck for now. Its late here, so I will try more after I get some sleep. Hopefully I can get it debugging tomorrow |
Got it running I think, I could not sleep and tried alot of different solutions from various sources on the internet. Could you confirm I am in the right place? Should I look in the console for logs, or is there somewhere else? I'm off sleep now, hopefully getting something running will ease my brain ;) |
Just the console, reproduce the issue a few times Open any log that has a button in the form of a triangle and take some screen shots |
No that is it. The player has already those two methods that are supposed to check if the player for some reason stops and restores things. But I didn't expect that crash, I will add an improvement to account for crashes, and wherever it happens the player will just restart the playback. I let you know when this is pushed to the app. Thank you great work! |
Nice to hear, and happy to be able to help with this. The app is a great service to us with a TV that doesnt have Twitch available by normal channels. I should be the one to thank you for your continued support to the project :) Looking forward to testing the app when the push has gone through. Thanks again! |
The above was deployed to the app if you have the release installed it must receive the update, just close the app if open and reopen it for it to receive it. The behavior now must be that the app will detect the problem and restart the player, but as I can't get this I don't know the behavior so it may just restart and work or something else. If the problem persists repeat the log pelase. |
Yes the player is gone there.... After that can you exit the player, then open a new stream and it plays fine? |
Some devices crash after playing ads for unknown reason. trying to get with player option causes it related to #277
Seems that the player is just gone, I have no idea how to recover from this error, and the biggest problem is that it crashes the app itself, which is almost impossible to achieve unless is an OS problem related to the player implementation on Tizen, which this problem is. The above separates it player initialization function, it may fix but if it doesn't I can have some more info trying to understand what to do next. please test again and share the logs even if work share it. |
OK, will see if I can improve this. Later I share another update. Thank you! |
No, thank you! I just had the same ad, and it managed to recover again. Will post more if I can figure out the condition for the first crash. For now, the fix is successful 2 out of 3 times :) |
The above must improve things, not 100% as this will probably never be 100% unless Samsung deals with the issues. Test and let me know the result. |
I havent had the correct ad to test the crashing yet, but I have had 2 random crashes without any logs being posted to the console with the last version. |
Very odd, the app closes the player before logging, maybe is the closing that crashes Will add some log to see if it's |
I'll be able to run more tests tomorrow, for now I need to get to sleeping, its past midnight here just now. |
Thank you no rush. I push the above just to be sure closing is crashing the app. When you have time to test let me know the result. |
Here is some link from Samsung documentation https://www.samsungdforum.com/b2b/Guide/GuideT3/TIZENH_GUIDE The above is very helpful for TV apps as it has various TV-specific helps This is the original Samsung Avplay documentation Anyone who wanna try something, can just download the main repo and modify the content of the app folder and install it just like explained on the main page but on the import step use the app folder not the extracted release. |
Describe the bug
Ads, starting Sep 1st have started to play, and when they finish the app fully crashes. When trying to relaunch the app, when you play a channel the same round of ads will play again, and again they crash. The only workaround is watching in an official app, watching the group of ads, rebooting the tv, and then watching again.
Now the ads don't always crash the app, but it does cause audio and video glitches for a short time after the ads play.
I don't have it installed with the debug tool, I will do that now and get a log tomorrow, but I felt this was important to get up.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
I expected no ads, as you have had it that way for many years now. But as long as I can watch without it crashing I'm happy
Device:
The text was updated successfully, but these errors were encountered: