search

TUM-Dev / gocast

TUMs lecture streaming service.
https://tum.live
MIT License
189 stars 42 forks source link

Ending and restarting a self stream with different resolutions causes transcoding to fail. #533

Open joschahenningsen opened 2 years ago

joschahenningsen commented 2 years ago

I had a lecturer stream in 2560x1440 for an hour, take a break and continue the lecture in 1920x1080. This messed up the .ts file and broke transcoding.

Ideally multiple parts of a stream should be saved in multiple files which are normalized before concatenating and transcoding them.

pczern commented 2 years ago

I think, I have the same error, can't play any streams on my iPad 15.6.1. Web inspector shows error when loading .ts files

joschahenningsen commented 2 years ago

Huh, do you have an example stream that doesn't work for you? Is it a livestream or video on demand?

pczern commented 2 years ago

when I am logged in on iPad, any recording / video on demand does not work. Only livestreams work when logged in on iPad. For example when trying to watch https://live.rbg.tum.de/w/ML/21009 on my iPad, there is no playback. Not sure if it is because of resolution, as it is happening with every recording when logged in.

Screenshot 2022-10-27 at 10 21 51
joschahenningsen commented 2 years ago

I'll have a thorough look this weekend, thanks for reporting! Do you have any insight as to why those request fail? Cors? Server error? Network error?

Which iPad and ipad os version do you use?

joschahenningsen commented 2 years ago

I was able to reproduce the issue on an iPad Pro running iPadOS 16. I'll try figure out the underlying issue. It's unrelated to this though.

pczern commented 2 years ago

Right now I use iPadOs 16.1, but I have the same issue since September 12th, where I used 15.6.1. It didn’t work on 15.7 too. I am using iPad Pro 11 inch 2021. Recordings fail in logged in state. In logged out state recordings work.

joschahenningsen commented 2 years ago

I created #752 to track this issue

SebiWrn commented 1 month ago

Should be fixed with #1233