[Approved] - Fix to "IMDB with AKA grabber" | Page 10

Discussion in 'Submit: Movie Info Grabbers' started by BalduinB, April 3, 2010.

  1. rivera
    • Premium Supporter

    rivera Retired Team Member

    Joined:
    December 1, 2008
    Messages:
    237
    Likes Received:
    21
    Ratings:
    +21 / 0
    Home Country:
    Russian Federation Russian Federation
    re: Fix to "IMDB with AKA grabber"

    That is fine, brother, but I wonder why these updates are not included in RC3...
    Still RC contains wrong script :(.

    Thank you very much for your job :)



    Just a small non-critical proposal.
    Red color usually stands for wrong items.
    So, may be you should use another color for it ?
    Assume, green color - which usually stands for OK.
     
  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. TLD

    TLD Portal Pro

    Joined:
    October 26, 2007
    Messages:
    608
    Likes Received:
    83
    Gender:
    Male
    Occupation:
    Retired
    Location:
    Rainy Washington
    Ratings:
    +86 / 0
    Home Country:
    United States of America United States of America
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Just wanted to stop by and say what a fantastic job you did fixing the IMDB feature of MP. I've been using MP for many years now and it has never worked better. I started using MP 1.1 RC2 a few weeks ago and have moved to MP 1.1 RC 3 now. with more than 500 movies IMDB only missed 3 and i have a lot of very old and obscure movies.

    Thank you very much.
     
  4. rivera
    • Premium Supporter

    rivera Retired Team Member

    Joined:
    December 1, 2008
    Messages:
    237
    Likes Received:
    21
    Ratings:
    +21 / 0
    Home Country:
    Russian Federation Russian Federation
  5. Deda
    • Team MediaPortal

    Deda Lead Dev MP1 Videos

    Joined:
    March 18, 2009
    Messages:
    2,423
    Likes Received:
    2,098
    Gender:
    Male
    Occupation:
    IT Consultant
    Location:
    Zagreb
    Ratings:
    +2,385 / 1
    Home Country:
    Croatia Croatia
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Sure, I will post here patch files (I just need a little time to see if everything is working as it should so not to change patch files every few minutes :) ).
    Concerning bugs in dll's and Configuration , there is not any. It's an upgrade or complete change of code and some cosmetics, because as it is now in normal release cover grabbing works in a very limited way and most of the time, results are very wrong (as I post earlier, whole thing which made me to do this is Alien (1979) movie for which I get only one and very funny cover and if I try to google it and let my 2 year old daughter randomly click on the screen she can't miss it :)).
    The bad thing in this is that patches touch essential files in MePo, but good thing is that they are not involved in any critical place inside Media Portal. So all add-in's, skins, media related changes, codecs, hardware or software setups doesn't even know about it. Otherwise, I wouldn't dare to do this (even for my personal usage).

    So, the fastest and clean patch is only IMDB csscript (which fixes more than movie details), but extra cover grabbing is more than a patch ( I would rather say it's an upgrade) and that is something what developers doesn't like right now in a "freeze state" of MP and I don't blame them for that.
     
  6. Dadeo
    • Premium Supporter

    Dadeo Docs Group Manager

    Joined:
    November 26, 2006
    Messages:
    5,340
    Likes Received:
    2,802
    Location:
    Himalayas, India
    Ratings:
    +3,053 / 0
    Home Country:
    Canada Canada
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Deda, I tested your script using RC3 files and it worked great for me. Do you have any objection if we just upload the script for now, and consider the rest of your great enhancments after the feature freeze. I think that is what you were saying, but I just wanted to check with you first.
     
  7. Deda
    • Team MediaPortal

    Deda Lead Dev MP1 Videos

    Joined:
    March 18, 2009
    Messages:
    2,423
    Likes Received:
    2,098
    Gender:
    Male
    Occupation:
    IT Consultant
    Location:
    Zagreb
    Ratings:
    +2,385 / 1
    Home Country:
    Croatia Croatia
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Sure, no problem about that.
    All in all, script itself also grabs film cover from IMDB as default one (the one you see up on the left on film page). Sometimes picture is not in very good quality but user can use current manual cover search and hope it will be right one).

    Just beware that script itself can't help if movie name contains country specific characters or symbols (method inside MP strips that and pass the movie name without it, except, I think, German and French characters) so there is a big chance that movie will not be on the list ie.:

    Lion King ½ -> Lion King
    Død Snøw -> Dd Snw

    Workaround is to use alternative title ie.:
    Hakuna Matata
    Dead Snow

    I fixed that in patched files, but without it, results can be unexpected.
     
  8. Dadeo
    • Premium Supporter

    Dadeo Docs Group Manager

    Joined:
    November 26, 2006
    Messages:
    5,340
    Likes Received:
    2,802
    Location:
    Himalayas, India
    Ratings:
    +3,053 / 0
    Home Country:
    Canada Canada
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Yep, all true and good points. But, when I use the script provided with RC3 on my 170 movies I get 33 conflicts (and thus manual intervention required) while with your script only 14 conflicts and those are all movies with two files. So 100% accuracy for me. PLUS I get descriptions for all movies while I get no descriptions with the RC3 script. The Rating info displays correctly with your script, which it does not with the RC3 IMDB Script.

    I did use your trick of using the alternate title to avoid special characters ;), but I am used to doing that as many scripts I use have the same problem.

    As you pointed out, you only get only one default cover when doing a scan, but when you use the IMDB lookup in Video Titles (F9 or Info/More Key) you get lots more nice quality covers with your script.

    My concern is only that I do not wish users to suffer just because we can not add the new features yet. So I thought a compromise was to at least add the script - then users can update it from within MP Config right away, and do not even need a new version of MP. Also, at the least a better script will be available/included in RC4 or MP 1.1.0 final. After which we hopfully can add all your great enhancements :D

    I just wanted your OK because as you said, users are not getting the full advantage of your patch by doing this, and I don't want them to think it is your fault!
     
  9. Deda
    • Team MediaPortal

    Deda Lead Dev MP1 Videos

    Joined:
    March 18, 2009
    Messages:
    2,423
    Likes Received:
    2,098
    Gender:
    Male
    Occupation:
    IT Consultant
    Location:
    Zagreb
    Ratings:
    +2,385 / 1
    Home Country:
    Croatia Croatia
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Yes, but only with patched files :) which includes IMDB Product group covers (new feature) and checked (by IMDB ttnumber) IMP Award covers (rewritten IMPAwardSearch code from begging). Otherwise, you will get only something from IMP Awards which now works as it takes first matching link and grabs covers from it.
     
  10. Dadeo
    • Premium Supporter

    Dadeo Docs Group Manager

    Joined:
    November 26, 2006
    Messages:
    5,340
    Likes Received:
    2,802
    Location:
    Himalayas, India
    Ratings:
    +3,053 / 0
    Home Country:
    Canada Canada
    Show System Specs
    re: Fix to "IMDB with AKA grabber"

    Thanks Deda, I updated the script on our server (so users will get it if they update Scripts) and added Mantis Issue 2923

    Chemelli has now submitted it to SVN 25902 so it will be in RC4. Thanks chemelli. And thank you Deda for this very valuable improvement to MP I hope we can get your new features added as soon as possible :D.
     
  11. Deda
    • Team MediaPortal

    Deda Lead Dev MP1 Videos

    Joined:
    March 18, 2009
    Messages:
    2,423
    Likes Received:
    2,098
    Gender:
    Male
    Occupation:
    IT Consultant
    Location:
    Zagreb
    Ratings:
    +2,385 / 1
    Home Country:
    Croatia Croatia
    Show System Specs
    • Like Like x 3
Loading...

Users Viewing Thread (Users: 0, Guests: 0)

  1. This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
    By continuing to use this site, you are consenting to our use of cookies.
    Dismiss Notice
  • About The Project

    The vision of the MediaPortal project is to create a free open source media centre application, which supports all advanced media centre functions, and is accessible to all Windows users.

    In reaching this goal we are working every day to make sure our software is one of the best.

             

  • Support MediaPortal!

    The team works very hard to make sure the community is running the best HTPC-software. We give away MediaPortal for free but hosting and software is not for us.

    Care to support our work with a few bucks? We'd really appreciate it!