Webcast List Handle iframe casts #403

Closed
phil-lopreiato opened this Issue Apr 20, 2015 · 20 comments

Projects

None yet

4 participants

@phil-lopreiato
Member

Since CMP Webcasts will be iframe embeds, we should make sure that non-deep linkable webcats are handled properly.

I think we should modify WebcastHelper to make all other webcasts type WEB_GAMEDAY, have them say "watch on the web", and link to http://www.thebluealliance.com/gameday#layout=0&view_0=eventKey

@1fish2
Contributor
1fish2 commented Apr 20, 2015
@phil-lopreiato
Member

It seems like that's possible also

So I guess we could do iframes that way. @fangeugene what exact types are the CMP webcasts going to be?

@fangeugene
Member

http://science.ksc.nasa.gov/robotics/first/st-louis-2015/

Last year I couldn't figure out any way to put them into gameday other than iframing them. (AKA it didnt follow our standard MMS or RTMP patterns)

@phil-lopreiato
Member

Wait, these are going to be flash streams, aren't they? Can't display those on mobile, so I guess kicking the user out to GameDay is the only thing we can do

@fangeugene
Member

Seems like it. Or just don't ship Webcasts for Champs if it's not going to work anyways.

@phil-lopreiato
Member

I'll have them display "Watch on GameDay" and open the web GameDay in the browser. That's easier than removing it, and it'll let users with flash (for some reason) still watch them.

@synth3tk
Member

They're using a m3u8 playlist, which means most likely HLS. This pleases the Android gods greatly. I'm not sure if the Gameday player can handle HLS yet, but I'll give it a go tonight or tomorrow.

I'm not familiar with Android development, but if we get the playlist URL for every field, you should be able to play it natively in the Android player, right?

nasam3u8

@phil-lopreiato
Member

Ooh, that's pretty nice. I think the default player can handle it fine, but I'd like to test it somehow.

We should (and @fangeugene) decide how that data will be encoded internally (make a new type, or just kept in an iframe), which will decide how it appears in the API, which drives how we display it on the app end.

@fangeugene
Member

If either of you can figure out how to do a new type, I'm down for it. Otherwise, I'll just create a few temporary pages and iframe those.

@phil-lopreiato
Member

@synth3tk if you can get GameDay to display m3u8 playlists, can you make it a new internal type, with channel being set as the playlist file? That way we can nicely display html5 video :)

@synth3tk
Member

I can definitely do that! Let me finish getting setup and I'll put something together.

For your tests, what better than a feed from NASA? Just clicking that link gives you the option to play it in a player, so I'm pretty sure we should be good to go, support-wise.
http://www.nasa.gov/multimedia/nasatv/NTV-Public-IPS.m3u8

@phil-lopreiato
Member

Cool! I'll play around with that. The only think I'll need from you is the format of the json dict you end up storing for the webcast. I propose something like:

{
 "type": "html5" (should just be able to render with <video> tag)
  "channe":  "http://www.nasa.gov/multimedia/nasatv/NTV-Public-IPS.m3u8" (or whatever)
}
@phil-lopreiato
Member

That was actually really easy. I implemented it in the app according to what I said above. If you change it, just post and I'll update.

@phil-lopreiato phil-lopreiato added a commit that referenced this issue Apr 21, 2015
@phil-lopreiato phil-lopreiato Add new webcast type: HTML5
Video will stream in native video player. For use with cmp 2015 streams,
which appear to be m38u playlists.

Fixes #403 and does a better job
ab8e99b
@phil-lopreiato phil-lopreiato added in progress and removed ready labels Apr 21, 2015
@synth3tk
Member

It'll probably just stick to that format above. Makes the most sense.

@phil-lopreiato
Member

👍

@1fish2
Contributor
1fish2 commented Apr 21, 2015

Awesome!

@synth3tk
Member

Running into issues integrating this desktop-side and I still haven't started packing. Thinking of adding an iframe item and just using that for the gameday site, leaving channel for the m3u8. That shouldn't break anything with your implementation, correct?

@phil-lopreiato
Member

I can make a case to handle that.

Or if you want, I might have a little more time to debug what you've got since I don't have to pack and catch a flight to STL :(

Up to you

@synth3tk
Member

I don't have to hit the road for another few hours, so I'll see what I can accomplish before then.

@synth3tk
Member

Got it! No changes need to be made to the formatting. About to commit in a few minutes.

@phil-lopreiato phil-lopreiato added a commit that closed this issue Apr 22, 2015
@phil-lopreiato phil-lopreiato Send iframe webcasts to GameDay web
Will close #403, but maybe not optimally
0241dcf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment