[VBS/Pitchfork/Crunchyroll] maruchan addons repository

  Thread Rating:
  • 1 Votes - 5 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Post Reply
cyberwizzard Offline
Member
Posts: 87
Joined: Dec 2008
Reputation: 6
Question    Post: #51
The plugin downloads the subs from CR, converts them and stores the result during playback on the hard drive (they are erased when playback stops).

Perhaps the ATV2 is not allowing write access to the location where the plugin expects to save the subtitles?

I am not familiar with the ATV2 but if XBMC is executed on top of a unix system, you might examine the directory structure and the permissions therein to see if the entire XBMC profile is writable.

On a side note: premium content is still broken - I will investigate this later on as I said before.
find quote
tarjan Offline
Junior Member
Posts: 29
Joined: Jun 2011
Reputation: 0
Post: #52
True, but it is running as a user that should have permission to everything. I'm not terribly familiar with python though, so I am not sure how to start poking around to see what files are generated where. I would have assumed in the cache directory and that seems to be good to go. It is generating other files there.
find quote
maruchan Offline
Fan
Posts: 311
Joined: Feb 2009
Reputation: 0
Post: #53
I pushed a quick update that seems to fix the basic Crunchyroll functionality for me. It only plays the lowest quality version of the streams, but it does seem to work on OSX and Windows 7. Those are the only systems I use, so no idea about ATV/Linux. Obviously, it requires the updated libRTMP.
find quote
cyberwizzard Offline
Member
Posts: 87
Joined: Dec 2008
Reputation: 6
Post: #54
maruchan Wrote:I pushed a quick update that seems to fix the basic Crunchyroll functionality for me. It only plays the lowest quality version of the streams, but it does seem to work on OSX and Windows 7. Those are the only systems I use, so no idea about ATV/Linux. Obviously, it requires the updated libRTMP.

I've been working on 2 issues:
Minor: show show name in listings so the recently added listing becomes useful instead of matching pictures Smile

Major: high def content for premium members. I got it working for pretty much everything, except that shows like Naruto and Bleach will not list because of a unicode error - I'm hoping that was fixed in the update Smile Edit: Yes it was fixed - yay!

Update: I do not have diff on Windows and my compare tool does not seem to generate standard diff files so here is the output of whatever I have.

Patch 1: Fix resolutions for premium members
Description: The Boxee feed is used to determine the available resolutions and it is reporting SD content for most episodes on CR while everything has at least a 480p version. So for now, simply overrule the reported resolutions so the user can set his/her preference. The server seems to have a fallback which returns a lower resolution if the requested one is not available.

Code:
[ 1 ]  C:\Users\Cyberwizzard\AppData\Roaming\XBMC\addons\plugin.video.crunchyroll\​resources\lib\crunchy_video.py
=======
82    *           #res_avail = resolutions.split(',')
      *           res_avail = resolutions.split(',')
83    <!          # As long as the Boxee feed incorrectly reports most media as being SD only, overrule the provided
84    <!          # resolution info: the server will stream the requested resolution or the next best thing.
85    <!          res_avail = [12, 20, 21]
=======

=======
113    *           playlist = "http://www.crunchyroll.com/xml/?req=RpcApiVideoPlayer_GetStandardConfig&media_id="+vid_id+"&video_format=102&video_quality="+stream['resolution']+"&auto_play=1&show_pop_out_controls=1&pop_out_disable_message=Only+All-Access+Members+and+Anime+Members+can+pop+out+this"
       *           playlist = "http://www.crunchyroll.com/xml/?req=RpcApiVideoPlayer_GetStandardConfig&media_id="+vid_id+"&video_format=102&video_quality=10&auto_play=1&show_pop_out_controls=1&pop_out_disable_message=Only+All-Access+Members+and+Anime+Members+can+pop+out+this"
=======

Patch 2: Add the series name in the episode listing. This helps the usability of the "Recently Added" section and also in the listings themselves. If you do not like it being on by default, it might be an idea to make it an option in the add-on.
Code:
[ 1 ]  C:\Users\Cyberwizzard\AppData\Roaming\XBMC\addons\plugin.video.crunchyroll\​resources\lib\crunchy_scraper.py
[ 2 ]  C:\Users\Cyberwizzard\AppData\Roaming\XBMC\addons\plugin.video.crunchyroll\​resources\lib\crunchy_scraper - Copy.py
=======
78    *              crunchy_main.UI().addItem({'Title':item['series_name']+' - '+item['name'], 'mode':'episode', 'Thumb':item['img'], 'id':item['ep_id'], 'page_url':item['page_url'], 'plot':item['description'], 'resolutions':item['resolutions']}, True, num_episodes)
      *              crunchy_main.UI().addItem({'Title':item['name'], 'mode':'episode', 'Thumb':item['img'], 'id':item['ep_id'], 'page_url':item['page_url'], 'plot':item['description'], 'resolutions':item['resolutions']}, True, num_episodes)
=======
(This post was last modified: 2011-07-20 00:03 by cyberwizzard.)
find quote
tarjan Offline
Junior Member
Posts: 29
Joined: Jun 2011
Reputation: 0
Post: #55
Cyber: I also have a fix for atv2 users. You were bang on in your first impression.

It seems that when the subtitle routine tries to write to special://temp it fails without giving any real information why. My guess is that it might be something where the original atv had a different directory (wiki states /Users/frontrow/.xbmc) which does not exist for the atv2.

I just hardcoded it to a directory I created in the python script and chown'd the directory to user mobile for testing. There is probably another easier solve, or this might just be a bug in the atv2 xbmc build but I was in QFE mode.

From: xbmc.Player().setSubtitles('special://temp/crunchy_'+mediaid+'.ass')
To: xbmc.Player().setSubtitles('/User/temp/crunchy_'+mediaid+'.ass')

There are a few references of this in crunchy_video.py which need to be changed.

Takes forever to start, but it works very well now. Might give your updates a shot as well later once I start my 2 week trial.
(This post was last modified: 2011-07-20 00:25 by tarjan.)
find quote
maruchan Offline
Fan
Posts: 311
Joined: Feb 2009
Reputation: 0
Post: #56
Just committed another update (2.2.1) featuring cyberwizzard's patches.
find quote
tarjan Offline
Junior Member
Posts: 29
Joined: Jun 2011
Reputation: 0
Post: #57
maruchan, any chance you could also update the subtitle open routine?

In reference:
http://forum.xbmc.org/showthread.php?p=8...post843098

The open('special://temp....) command doesn't seem to work on the ATV2 and amet suggested a replacement of using

subfile = open(xbmc.translatePath('special://temp/crunchy_'+mediaid+'.ass'), 'w')

Looking at it, I can't think of a reason why this wouldn't work on all platforms. If anyone else wants to give it a shot to verify, that would be very appreciated.
find quote
spiff Offline
Grumpy Bastard Developer
Posts: 12,171
Joined: Nov 2003
Reputation: 81
Post: #58
it will work, and is in fact required on all platforms in master. we don't inject the vfs any longer, so paths have to be translated.

Always read the XBMC online-manual, FAQ and search the forum before posting.
Do not e-mail XBMC-Team members directly asking for support. Read/follow the forum rules.
For troubleshooting and bug reporting please make sure you read this first.
find quote
maruchan Offline
Fan
Posts: 311
Joined: Feb 2009
Reputation: 0
Post: #59
OK, I replaced the direct paths with xbmc.translatePath in version 2.2.2. Let me know if that fixes things.
find quote
theredbaron Offline
Junior Member
Posts: 4
Joined: Aug 2011
Reputation: 0
Post: #60
Do you have to have a prem crunchyroll account to use this? Cause I can't get the video's to work. Instead of "loading stream" it loads directory, and then does nothing.
find quote
Post Reply