+- XBMC Community Forum (http://forum.xbmc.org)
+-- Forum: Help and Support (/forumdisplay.php?fid=33)
+--- Forum: Add-ons Help and Support (/forumdisplay.php?fid=27)
+---- Forum: Third Party Repositories (/forumdisplay.php?fid=157)
+---- Thread: [VBS/Pitchfork/Crunchyroll] maruchan addons repository (/showthread.php?tid=83795)
- cyberwizzard - 2011-07-19 13:00
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.
- tarjan - 2011-07-19 15:55
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.
- maruchan - 2011-07-19 23:10
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.
- cyberwizzard - 2011-07-19 23:34
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
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
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)
=======- tarjan - 2011-07-20 00:20
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.
- maruchan - 2011-07-20 01:09
Just committed another update (2.2.1) featuring cyberwizzard's patches.
- tarjan - 2011-07-20 16:24
maruchan, any chance you could also update the subtitle open routine?
In reference:
http://forum.xbmc.org/showthread.php?p=843098#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.
- spiff - 2011-07-20 16:37
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.
- maruchan - 2011-07-20 17:03
OK, I replaced the direct paths with xbmc.translatePath in version 2.2.2. Let me know if that fixes things.
- theredbaron - 2011-08-18 23:02
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.