Conversation
|
This looks really nice! I assume the answer is yes but just want to confirm that this works for you? |
Thanks! And yep, it works nicely for me: |
|
So I am using offline mode (pre-downloaded the videos). The screensaver is refusing to start with this in kodi.log: Missing some stuff it seems |
|
Ahh okay, I haven't tested offline mode. I guess if you stay offline it can't re-get the file and extract the plist file so it can build stuff. I can add a check so it just doesn't create the subtitle info if it can't find the file with the descriptions. |
|
Can you easily adapt it to fetch the data it seems when in offline mode? If too much, can I manually wget what it needs? |
I need to check what is available in this case. Does the You can wget |
|
Man I downloaded these ages ago. If you have a fast connect, go ahead and try offline mode. It's probably more efficient for you to replicate it on your side. |
I did this but still got errors: |
|
I understand the problem, it's more a question of the "right" solution. I.e. if you're in offline mode, the addon probably shouldn't go re-download a bunch of stuff. I do realize that |
|
Maybe add a check for the needed files for users with an offline set of videos already and grab it if not present? Also, can you see my error one post above. After manually placing that directory, it is still erroring out. |
Hrmm, somehow your Perhaps your entries.json is way out of date? |
|
I did enable offline mode and download a couple of videos, and it's all working, but obviously I have the up to date files at this point |
|
If you could post an entry or two from your |
|
Sure here is the entire file: https://gist.github.com/graysky2/68838af1a34bab64c9c094ae310f10fd Can you post yours? I am happy to just overwrite it. |
|
Ahh yeah, you have entries like: {
"accessibilityLabel": "California to Vegas",
"categories": [
"55B7C95D-CEAF-4FD8-ADEF-F5BC657D8F6D"
],
"clipNumber": "6",
"official": "kubedzero_manually_added",
"url-1080-HDR": "https://sylvan.apple.com/Aerials/2x/Videos/comp_GMT306_139NC_139J_3066_CALI_TO_VEGAS_v07_HDR_PS_FINAL_22062018_HDR_2K_HEVC.mov",
"url-1080-SDR": "https://sylvan.apple.com/Aerials/2x/Videos/comp_GMT306_139NC_139J_3066_CALI_TO_VEGAS_v07_SDR_FINAL_22062018_SDR_2K_HEVC.mov",
"url-4K-HDR": "https://sylvan.apple.com/Aerials/2x/Videos/comp_GMT306_139NC_139J_3066_CALI_TO_VEGAS_v07_HDR_PS_FINAL_22062018_HDR_4K_HEVC.mov",
"url-4K-SDR": "https://sylvan.apple.com/Aerials/2x/Videos/comp_GMT306_139NC_139J_3066_CALI_TO_VEGAS_v07_SDR_FINAL_22062018_SDR_4K_HEVC.mov"
},which don't have But I'm not 100% sure if it will match the videos you already have downloaded |
|
Should I just delete the downloaded videos and re-download them? |
|
I mean you could get all the latest videos easily yeah by just going into online mode and downloading new videos :) |
|
OK so as I test I will re-download everything after applying your new commit. That should grab the tarball I manually downloaded too, yes? |
Yeah, in online mode it always downloads that tar and extracts the latest (at least afaict, I only just started looking at this code today ;) ) |
|
I can't get it working even in offline mode. Videos just play but no subtitles even though they are enabled. The installation was just applying the PR and copying the entire dirtree to Is that all? When I start the addon with offline disabled: |
Where exactly should it download that tarball? I am running the latest version of this PR in offline mode and cannot find anything in
That is the file I am using.
It does not have any entries for that ... does your copy? If so, how? On Arch Linux, this code is provided by kodi-addon-screensaver-apple-aerial which I maintain. Basically, the PKGBUILD just downloads this code, applies this PR, and copies the contents of this repo to |
|
Hrmm. I can probably look at this again next week. Just to check, if you can could you just fully uninstall your current version including the user data and then install, does it work? I suspect somehow the "old" data is hanging around and not getting updated with the new needed stuff. I have an arch machine so I'll test out the AUR package as soon as I have a minute. Very cool that you want to package this :) BTW, is this repo "abandoned"? Seems like stuff doesn't really get merged. |
|
Just going by the last commit, @enen92 might not care to continue the project. You can carry the torch with your fork. I will try clearing all user data, but yes, using pacman guarantees none of the old file are present. |
|
So I did have a settings.xml in |
|
@nicklan - did you get a chance to check this out? |
2 participants
The resources tar actually includes a plist file that includes descriptions about each video.
Within that file there can be a single description for the whole video, or multiple descriptions. They are keyed by the number of seconds into the video that the description is applicable. The keys are referenced from the
entries.jsonin thepointsOfInterestelement.This PR does the following main things:
When downloading/parsing the video info and building the playlist:
poislist based on thepointsOfInterestelement in entries.json and indexing into the above fileWhen playing videos, if descriptions are enabled
poiselement into a.srttemp subtitle file