LinuxQuestions.org
Register a domain and help support LQ
Go Back   LinuxQuestions.org > Forums > Linux Forums > Linux - Distributions > Fedora
User Name
Password
Fedora This forum is for the discussion of the Fedora Project.

Notices



Reply
 
Search this Thread
Old 03-28-2013, 01:42 PM   #3841
BostonPeng
Member
 
Registered: Jul 2009
Location: Boston, MA
Distribution: SolydK Testing + KDE 4.9.5
Posts: 88

Rep: Reputation: Disabled

Quote:
Originally Posted by daftcat View Post
BostonPeng, see post above. Leaving user= and pass= blank will get you access to the 1200K condensed game stream and the highlights.

Dov, changing free_condensed to use_librtmp will only work for that one case but it will break all the subscriber use cases including nexdef which has nothing to do with librtmp. I will have to look deeper into how to re-work the logic for librtmp. It will probably go into prepareMediaPlayerCmd effectively removing the need for that "if" statement. But I probably can't do that today as it needs a bit of testing.

In the meantime, I can disable it in the code and release note it until I have it re-worked.

I'm heading out of town soon for the weekend so I'm likely not going to take new patches until next week. Non-subscribers have a workaround by leaving user= and pass= blank and librtmp users can revert back to rtmpdump until I have this sorted.
ZANG!!! I'm writing my article now so I'll include that little tidbit (and change my config file). Should I point people to get the latest svn code or just tell them to get the 2013 release?
 
Old 03-28-2013, 01:52 PM   #3842
dovf
LQ Newbie
 
Registered: Jul 2010
Posts: 13

Rep: Reputation: 1
Quote:
Originally Posted by daftcat View Post
Dov, changing free_condensed to use_librtmp will only work for that one case but it will break all the subscriber use cases including nexdef which has nothing to do with librtmp. I will have to look deeper into how to re-work the logic for librtmp. It will probably go into prepareMediaPlayerCmd effectively removing the need for that "if" statement. But I probably can't do that today as it needs a bit of testing.

In the meantime, I can disable it in the code and release note it until I have it re-worked.

I'm heading out of town soon for the weekend so I'm likely not going to take new patches until next week. Non-subscribers have a workaround by leaving user= and pass= blank and librtmp users can revert back to rtmpdump until I have this sorted.
Sure, no sweat! It doesn't seem like this would really affect many users, so definitely better to wait until you are able to work it out correctly. I may take a deeper look at the code myself, and maybe suggest further changes (though it's hard if I can't test the main use-case) -- I'm glad that you have no problem just pointing out when the "fix" is wrong .
 
Old 03-28-2013, 02:13 PM   #3843
BostonPeng
Member
 
Registered: Jul 2009
Location: Boston, MA
Distribution: SolydK Testing + KDE 4.9.5
Posts: 88

Rep: Reputation: Disabled
Quote:
Originally Posted by daftcat View Post
BostonPeng, see post above. Leaving user= and pass= blank will get you access to the 1200K condensed game stream and the highlights.
I just tried it without login info and I could only get the 550K condensed game for yesterday's Nats v Cards game. Should I try a different game?
 
Old 03-28-2013, 02:29 PM   #3844
dovf
LQ Newbie
 
Registered: Jul 2010
Posts: 13

Rep: Reputation: 1
Quote:
Originally Posted by BostonPeng View Post
I just tried it without login info and I could only get the 550K condensed game for yesterday's Nats v Cards game. Should I try a different game?
Have you set free_condensed back to 0?
 
Old 03-28-2013, 02:35 PM   #3845
BostonPeng
Member
 
Registered: Jul 2009
Location: Boston, MA
Distribution: SolydK Testing + KDE 4.9.5
Posts: 88

Rep: Reputation: Disabled
Quote:
Originally Posted by dovf View Post
Have you set free_condensed back to 0?
Thanks, dovf, I completely missed that.

I'm writing my article and I realized that the mlbhls site doesn't have any downloads available anymore. I'm going back through the older posts here but can someone let me know how I should tell my readers to get it? I still had the code from last year so I didn't realize this issue had arisen.
 
Old 03-28-2013, 02:40 PM   #3846
dovf
LQ Newbie
 
Registered: Jul 2010
Posts: 13

Rep: Reputation: 1
Quote:
Originally Posted by BostonPeng View Post
I'm writing my article and I realized that the mlbhls site doesn't have any downloads available anymore. I'm going back through the older posts here but can someone let me know how I should tell my readers to get it? I still had the code from last year so I didn't realize this issue had arisen.
See the REQUIREMENTS file in mlbviewer -- it seems like you need to download mlbhls directly from the experimental branch on svn. (I have no experience with this, just happened to notice this in the REQUIREMENTS file today...).
 
Old 03-28-2013, 02:42 PM   #3847
BostonPeng
Member
 
Registered: Jul 2009
Location: Boston, MA
Distribution: SolydK Testing + KDE 4.9.5
Posts: 88

Rep: Reputation: Disabled
Quote:
Originally Posted by dovf View Post
See the REQUIREMENTS file in mlbviewer -- it seems like you need to download mlbhls directly from the experimental branch on svn. (I have no experience with this, just happened to notice this in the REQUIREMENTS file today...).
That's what I get for having so many text files open at once. Thanks again.
 
Old 03-28-2013, 03:50 PM   #3848
daftcat
mlbviewer Maintainer
 
Registered: Apr 2008
Posts: 1,767

Rep: Reputation: 76
Quote:
Originally Posted by dovf View Post
Sure, no sweat! It doesn't seem like this would really affect many users, so definitely better to wait until you are able to work it out correctly. I may take a deeper look at the code myself, and maybe suggest further changes (though it's hard if I can't test the main use-case) -- I'm glad that you have no problem just pointing out when the "fix" is wrong .
So I did a little bit of snooping and testing and it looks like, except for that one case, librtmp is actually there in the code and working (sort of) as expected. I may have to recompile my mplayer2 to get some of the streams working correctly, but I can see that the code is at least trying to format the command string correctly.

So I think what I'll end up doing is deprecating free_condensed and update the documentation.

I'm not too concerned about the login issue at startup with null user and pass. I'm glad it lets you continue anyway. But a more permanent fix would be to handle it in MLBSession (something like return None if user/pass blank.) A more elegant fix would be to re-implement the DISABLED_FEATURES list and keep non-subscribers away from those features I know won't work like game audio and game video. This way we won't nag their servers with null logins and requests we know will fail.

Hopefully you find the code a little more readable , a bit better organized after the rewrite (except maybe the media parsing which was largely untouched.)
 
Old 03-28-2013, 06:19 PM   #3849
chrisVV
Member
 
Registered: Aug 2010
Posts: 110

Rep: Reputation: 1
Quote:
Originally Posted by daftcat View Post
[snip]
1. Python2.7 (or newer, but not Python3.x) - no other python libraries to install
2. Mplayer2 or VLC
3. rtmpdump for basic and subscriber condensed games
4. mlbhls for nexdef

3. and 4. not necessary for non-subscribers.
3 is misleading. mlbhls works fine for both basic and premium (but you need the use_wired_web option for basic). rtmpdump also works for basic, as it presumably does also for premium.
 
Old 03-28-2013, 08:09 PM   #3850
daftcat
mlbviewer Maintainer
 
Registered: Apr 2008
Posts: 1,767

Rep: Reputation: 76
Quote:
Originally Posted by chrisVV View Post
3 is misleading. mlbhls works fine for both basic and premium (but you need the use_wired_web option for basic). rtmpdump also works for basic, as it presumably does also for premium.
Let me rephrase that.

mlbhls is required for nexdef streams
rtmpdump is required for non-nexdef streams and condensed games

Better?
 
Old 03-28-2013, 08:43 PM   #3851
daftcat
mlbviewer Maintainer
 
Registered: Apr 2008
Posts: 1,767

Rep: Reputation: 76
free_condensed has been deprecated (I removed the selection of the 3GP stream and override it to False for good measure after config file is read) and the documentation has been updated with regards to leaving user= and pass= blank for non-subscribers.

I haven't fixed librtmp yet nor have I fixed the login error. But for now, if nothing changes between now and Opening Day, I would consider this a pretty good Opening Day release.

It always gets more testing and more bugs/errors after Spring Training, anyway.

Now I really have to leave for airport but I'll have plenty of time on their wifi to see if Sourceforge has their problems sorted enough to update the download tarball.
 
Old 03-29-2013, 05:58 AM   #3852
dovf
LQ Newbie
 
Registered: Jul 2010
Posts: 13

Rep: Reputation: 1
Quote:
Originally Posted by daftcat View Post
free_condensed has been deprecated (I removed the selection of the 3GP stream and override it to False for good measure after config file is read) and the documentation has been updated with regards to leaving user= and pass= blank for non-subscribers.

I haven't fixed librtmp yet nor have I fixed the login error. But for now, if nothing changes between now and Opening Day, I would consider this a pretty good Opening Day release.

It always gets more testing and more bugs/errors after Spring Training, anyway.

Now I really have to leave for airport but I'll have plenty of time on their wifi to see if Sourceforge has their problems sorted enough to update the download tarball.
Thanks, daftcat! And yes, I certainly don't think that any of the issues I'm raising --- as they are today, unaddressed --- would change this from being a great Opening Day release!

So, for when you have time to get back to this:

1. How come you left any mention at all of free_condensed in the code -- why not just remove it completely?
2. I'm revisiting my suggested fix from yesterday regarding librtmp (below, modified against the latest changes): note that this would only affect users who have use_librtmp=1; given your comment in the docs about librtmp: "this may not be working yet in 2013. If you want it, ask for it...", I would expect that anyone having use_librtmp=1 is already in a gray zone of maybe working, maybe not? BTW, what is the relation between nexdef and librtmp -- are they mutually exclusive, or not necessarily?
3. Do you have any objections to my publishing a mercurial mirror of mlbviewer on bitbucket?

Code:
diff -r 5be6d992bdbb mlbviewer.py
--- a/mlbviewer.py  Fri Mar 29 00:39:09 2013 +0000
+++ b/mlbviewer.py  Fri Mar 29 11:10:24 2013 +0300
@@ -496,7 +496,8 @@
                     myscr.refresh()
                     time.sleep(2)
                     continue
-                mediaUrl = mediaStream.prepareMediaPlayer(mediaUrl)
+                if not mycfg.get('use_librtmp'):
+                    mediaUrl = mediaStream.prepareMediaPlayer(mediaUrl)
                 eventId  = available[listwin.current_cursor][6]

             cmdStr = mediaStream.preparePlayerCmd(mediaUrl, eventId,streamtype)
Meanwhile, have a safe flight and nice weekend!
 
Old 03-29-2013, 03:10 PM   #3853
chrisVV
Member
 
Registered: Aug 2010
Posts: 110

Rep: Reputation: 1
Quote:
Originally Posted by daftcat View Post
Let me rephrase that.

mlbhls is required for nexdef streams
rtmpdump is required for non-nexdef streams and condensed games

Better?
Looks fine to me.
 
Old 03-29-2013, 10:18 PM   #3854
daftcat
mlbviewer Maintainer
 
Registered: Apr 2008
Posts: 1,767

Rep: Reputation: 76
Quote:
Originally Posted by dovf View Post
Thanks, daftcat! And yes, I certainly don't think that any of the issues I'm raising --- as they are today, unaddressed --- would change this from being a great Opening Day release!

So, for when you have time to get back to this:

1. How come you left any mention at all of free_condensed in the code -- why not just remove it completely?
2. I'm revisiting my suggested fix from yesterday regarding librtmp (below, modified against the latest changes): note that this would only affect users who have use_librtmp=1; given your comment in the docs about librtmp: "this may not be working yet in 2013. If you want it, ask for it...", I would expect that anyone having use_librtmp=1 is already in a gray zone of maybe working, maybe not? BTW, what is the relation between nexdef and librtmp -- are they mutually exclusive, or not necessarily?
3. Do you have any objections to my publishing a mercurial mirror of mlbviewer on bitbucket?

Code:
diff -r 5be6d992bdbb mlbviewer.py
--- a/mlbviewer.py  Fri Mar 29 00:39:09 2013 +0000
+++ b/mlbviewer.py  Fri Mar 29 11:10:24 2013 +0300
@@ -496,7 +496,8 @@
                     myscr.refresh()
                     time.sleep(2)
                     continue
-                mediaUrl = mediaStream.prepareMediaPlayer(mediaUrl)
+                if not mycfg.get('use_librtmp'):
+                    mediaUrl = mediaStream.prepareMediaPlayer(mediaUrl)
                 eventId  = available[listwin.current_cursor][6]

             cmdStr = mediaStream.preparePlayerCmd(mediaUrl, eventId,streamtype)
Meanwhile, have a safe flight and nice weekend!
1. Quick fix was to deprecate it (e.g. remove the difference in stream selection) and an added safety was to override it after cfg file is read. The only reason why I didn't remove it completely is because I put it some if/elif/else code that I haven't had a chance to de-tangle.

2. I don't know if I want to put that patch in only because I haven't had the chance to test all the use cases and the bug seems to only affect condensed games. There is code elsewhere (presumably locateMedia() of which locateCondensedMedia() is an early shortcut within that procedure) that handles librtmp.

In other words, your use case is much smaller compared to other cases that can be affected by your patch.

In more blunt terms, the patch can do more harm than the bug.

At this point, I would say if you can use rtmpdump and you cannot justify to me why you would rather not use rtmpdump, wait it out until I have the time to put a proper fix in.

2a. librtmp/rtmpdump and nexdef are mutually exclusive except condensed games. There are no nexdef streams for condensed games. Rtmpdump or librtmp is exclusively used for condensed games unless you want the lower quality 3GP streams.

3. I don't know what this means to have a mercurial mirror on bitbucket. Does it require any extra effort on my part? Could it result in different versions of the code residing on the mirror than what's on sourceforge? Will there be different versioning schemes? Again, like the patch above, it sounds like it could result in more work for me than it's worth for everyone else. It's GPL so you can do with it what you want but I'm not likely to do anything but maintain the sourceforge versions.
 
Old 03-30-2013, 06:06 PM   #3855
chrisVV
Member
 
Registered: Aug 2010
Posts: 110

Rep: Reputation: 1
Interface crashes

If anyone has been having their game interface crashing these last few days, this patch seems to fix it. When the patch is applied, it might apply a line or two off, as I have also patched the file to allow 800kb/s streams which MLB.TV are providing that on live streams this season.

The patch might also come in handy for the all star game.
Attached Files
File Type: txt interface.txt (1.1 KB, 3 views)
 
  


Reply

Tags
help, install, installation, instructions, seek, vlc, windows


Thread Tools Search this Thread
Search this Thread:

Advanced Search

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is Off
HTML code is Off


Similar Threads
Thread Thread Starter Forum Replies Last Post
mlb.com gameday audio stream statmobile Linux - Newbie 6 05-06-2008 11:16 PM
link dies intermittently-seemingly at random- between win<->linux not linux<->linux?? takahaya Linux - Networking 10 03-09-2007 11:37 PM
triple boot linux/linux/linux No Windows involved toastermaker Linux - Newbie 12 03-02-2006 11:40 PM
Redhat (rhel v2.1) bootup problem with linux (linux vs linux-up) namgor Linux - Software 2 06-24-2004 03:49 PM


All times are GMT -5. The time now is 07:33 AM.

Main Menu
Advertisement
My LQ
Write for LQ
LinuxQuestions.org is looking for people interested in writing Editorials, Articles, Reviews, and more. If you'd like to contribute content, let us know.
Main Menu
Syndicate
RSS1  Latest Threads
RSS1  LQ News
Twitter: @linuxquestions
identi.ca: @linuxquestions
Facebook: linuxquestions Google+: linuxquestions
Open Source Consulting | Domain Registration