Release TheAudioDb.com Music Video Scraper - Printable Version +- Kodi Community Forum (https://forum.kodi.tv) +-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33) +--- Forum: Add-on Support (https://forum.kodi.tv/forumdisplay.php?fid=27) +---- Forum: Information Providers (scrapers) (https://forum.kodi.tv/forumdisplay.php?fid=147) +----- Forum: Music Video Scrapers (https://forum.kodi.tv/forumdisplay.php?fid=304) +----- Thread: Release TheAudioDb.com Music Video Scraper (/showthread.php?tid=136105) |
TheAudioDb.com Music Video Scraper - olympia - 2012-07-14 Music Video Scraper Source: www.theaudiodb.com ( links) Code: https://github.com/zag2me/metadata.musicvideos.theaudiodb.com Status: Working Notes: The scraper ONLY supports music video files with the naming convention: 'artist - track'. -> Note the dash and the space between and after!!! As being said it will not work perfectly, but I do hope it does the most... See more about theaudiodb.com in this thread: http://forum.xbmc.org/showthread.php?tid=134260 Adding Music Videos Sign up and log in on www.theaudiodb.com site and search for the track. Or navigate through artist name >> album >> Track. Then edit the track details and enter a youtube link (preferably vevo) into the correct text box. Also add some music video screenshots if you like. RE: [Release] TheAudioDb.com Music Video Scraper - Domin - 2012-08-04 I get the following error trying to scrape for videos: Code: 11:51:36 T:139888797792064 DEBUG: ------ Window Init (DialogVideoScan.xml) ------ The scanning then stops and says the server is unavailable and i can then press yes to continue scanning, if i press yes it will do the same again, and if i press no it will just quit. Hope this can be fixed, and thanks for working on an mvid plugin ;-) Regards Domin RE: [Release] TheAudioDb.com Music Video Scraper - RiotGrrl - 2012-09-25 The scraper doesn't match any songs with a hyphen in. eg. Placebo - Infra-red, Gorillaz - 19-2000, Nirvana - Heart-shaped Box, etc. I've tried just about everything - omitting the hyphen, a double hyphen "--", a space, underscore, etc - but nothing works. Is there a wildcard character that i could use to search? Would appreciate any help, I can't find any threads that solve this. RE: [Release] TheAudioDb.com Music Video Scraper - clackerdacker - 2012-11-07 (2012-08-04, 12:14)Domin Wrote: I get the following error trying to scrape for videos: Same here. Any clues? RE: [Release] TheAudioDb.com Music Video Scraper - zag - 2013-01-16 Quote:Correctly matched 1 of 24 music videos to its artist. Doesn't like dashes, searching "B.o.B." results in an "unable to connect to remote server" error. Olympia could you take a look at these issues maybe? The dash thing is a big problem as it effects lots of searches. Code: http://www.theaudiodb.com/api/v1/json/1/searchtrack.php?s=placebo&t=Infra-red Works so it must be scraper problem. The issue with B.o.B is that there is an extra fullstop at the end of the users search string. It shouldn't cause a server error though. Code: http://www.theaudiodb.com/api/v1/json/1/searchtrack.php?s=b.o.b.&t=arena RE: [Release] TheAudioDb.com Music Video Scraper - skypichat - 2013-01-20 Hello, Possible to read ID3 tag from MP4 files ? The scrapper disconnect if the name contain "__ without "-" RE: [Release] TheAudioDb.com Music Video Scraper - zag - 2013-01-21 (2013-01-20, 13:16)skypichat Wrote: Hello, This is a metadata scraper, it doesn't read ID3 tags. Try checking the database with sqlite to see how its been added to xbmc RE: [Release] TheAudioDb.com Music Video Scraper - n1md4 - 2013-01-26 Hi. I've added this today, and get the same error: Code: 13:07:53 T:774605632 NOTICE: Previous line repeats 2 times. RE: [Release] TheAudioDb.com Music Video Scraper - olympia - 2013-01-26 you need ' - ' between the artist and the track. RE: [Release] TheAudioDb.com Music Video Scraper - n1md4 - 2013-01-26 Wizard! You're right, working now. RE: [Release] TheAudioDb.com Music Video Scraper - zag - 2013-03-27 I'll be looking into some heuristic search methods soon for www.theaudiodb.com API in the hope of improving this scraper. Can people list some examples of how their music videos are named please? RE: [Release] TheAudioDb.com Music Video Scraper - bradford9999 - 2013-03-27 When I tried to import my music videos, I noticed that the current scraper has an exact way of handling filenames. It MUST be ARTIST - SONGNAME.ext. Note the "space dash space" in between artist and songname. If there aren't spaces surrounding the dash, it won't work. Beastie Boys - Sabatoge.mpg -- works Beastie Boys-Sabatoge.mpg -- fails Note: you must have the exact artist and song, spelled correctly, including punctuation and spacing. panic at the disco - i write sins not tragedies.mpg -- fails panic! at the disco - i write sins not tragedies.mpg -- works Taking back sunday - This photograph is proof.mpg -- fails Taking back sunday - This photograph is proof(I know you know).mpg -- fails Taking back sunday - This photograph is proof (I know you know).mpg -- works Problems with the scraper:
Example: the all-american rejects - move along.mpg -- fails Bowling for Soup - 1985.mpg -- fails. I haven't figured out a way around these two errors. I suspect you'll also get an error if the artist name is all numeric, but I can't be certain. It is NOT an issue with Audiodb's scraping service. If you were to search for my above examples, it would return json: http://www.theaudiodb.com/api/v1/json/1/searchtrack.php?s=Bowling%20for%20soup&t=1985 http://www.theaudiodb.com/api/v1/json/1/searchtrack.php?s=the%20all-american%20rejects&t=move%20along XBMC'x addon is not parsing the values correctly when it finds all numeric songs or dashes in the artist name or song. Hopefully this is helpful to someone. I'm going to continue finding out if there is a way around this. Maybe change the separator to a pipe (|)? I don't imagine many songs/artists have that in their name. Finally---if the scraper fails to find information, the error displayed will be "Cannot connect to server." Misleading, because it could connect, it just couldn't find your files based on what it parsed. RE: [Release] TheAudioDb.com Music Video Scraper - scudlee - 2013-03-27 The issue with dashes is a fairly easy (two-fold) fix. The first issue is a small piece of code in XBMC, that handles how the "title" of a video is passed to the scraper's CreateSearchUrl (for music videos the title should just be the filename). For TV shows and Movies it's done in two passes, the first pass leaves dashes intact, if that fails to return any results the second pass is done with the dashes replaced by spaces. For Music Videos, it basically jumps straight to the second pass, removing the dashes straight away. (See this line in Scraper.cpp). Removing that restriction in the code will only get you halfway there, though. Because that restriction exists, the scraper doesn't actually split the file name into artist and song by looking for the "space dash space", it splits it on "space space space", because the dash has been replaced before the scraper sees it (you can test this yourself by renaming a file to have three spaces between artist and song - it will still work). Without updating the scraper also, all that will happen is that the scraper will look for the triple space on the first pass and fail to find it, and so fall back to the second pass, resulting in identical behaviour. Basically, the "Or is a music video" code needs removing from Scraper.cpp and the scraper needs updating to split on "space dash space" (%20-%20) (and "space space space" for backwards compatibility). RE: [Release] TheAudioDb.com Music Video Scraper - bradford9999 - 2013-03-27 This is great! Thanks for the tips! Is this something that we as users can contribute to, or is the repo locked to admins? I'm new to the XBMC community, so I'm not sure how to fix this for everyone, or if we are just supposed to change this locally. RE: [Release] TheAudioDb.com Music Video Scraper - scudlee - 2013-03-27 If you can compile your own copy of XBMC, you can edit the code (just remove the " || Content() == CONTENT_MUSICVIDEOS") and then also replace the %20%20%20 with %20-%20 in your local copy of the scraper xml. Note: I haven't actually tested this yet, but it seems solid. When I do get the chance to test it, (if it works) I'll do a pull request, and when/if that gets pulled, I will update the scraper accordingly for everyone (but obviously you'd need to be running a nightly build from after the pull request gets pulled for it to work properly). |